Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[backport 8.x] Fix CI via updating versions of Rails tested & Solr 9.8 fix #3500

Merged
merged 3 commits into from
Jan 29, 2025

Conversation

seanaery
Copy link
Contributor

Note: This PR's primary aim is to get the test suite passing again in CI for the release-8.x branch, especially to facilitate future backports. Rails versions below 7.1 are all now EOL. There's a bug at the Rails core level in pre-7.1 (see #3493) that complicates testing. This PR removes the last vestiges of pre-7.1 versions from the CI matrix for Blacklight 8.x but still retains existing >= 6.1 compatibility via the .gemspec.

Implementers using Blacklight 8 with EOL Rails < 7.1 may need to pin the concurrent-ruby gem to < 1.3.5

sandbergja and others added 3 commits January 29, 2025 14:37
…r 9.8 compatibility

As a result of https://issues.apache.org/jira/browse/SOLR-16781,
<lib> entries are now ignored by default.  The Solr guide
recommends adding them to the solr.modules system property or the
SOLR_MODULES envvar instead:
https://solr.apache.org/guide/solr/latest/configuration-guide/solr-modules.html

Closes #3496 and helps to unbreak CI
- Use latest 8.0, 7.2, 7.1 releases
- Stop testing 6.x
Copy link
Contributor

@sandbergja sandbergja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much, @seanaery !

@sandbergja sandbergja merged commit 2559144 into release-8.x Jan 29, 2025
13 checks passed
@sandbergja sandbergja deleted the backports-fix-ci-8.x branch January 29, 2025 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants