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

Lock concurrent-ruby gem to fix CI #180

Merged
merged 1 commit into from
Jan 26, 2025

Conversation

tagliala
Copy link
Contributor

To address compatibility issues with Rails 7.0, this commit locks concurrent-ruby gem to version 1.3.4.

This change is necessary to prevent an uninitialized constant error related to ActiveSupport::LoggerThreadSafeLevel::Logger, as documented in issue rails/rails#54260.

By specifying this version, we ensure that our CI integration specs pass consistently, providing a stable development and testing environment.

This measure will remain in place until the integration tests upgrade to Rails 7.1, which involves significant changes that require careful consideration and testing.

To address compatibility issues with Rails 7.0, this commit locks
`concurrent-ruby` gem to version 1.3.4.

This change is necessary to prevent an uninitialized constant error
related to `ActiveSupport::LoggerThreadSafeLevel::Logger`, as documented
in issue rails/rails#54260.

By specifying this version, we ensure that our CI integration specs pass
consistently, providing a stable development and testing environment.

This measure will remain in place until the integration tests upgrade
to Rails 7.1, which involves significant changes that require careful
consideration and testing.
@tagliala tagliala force-pushed the chore/fix-integration-ci branch from 4038a54 to 69c4095 Compare January 25, 2025 12:51
@tagliala tagliala mentioned this pull request Jan 25, 2025
@drwl
Copy link
Owner

drwl commented Jan 26, 2025

Thank you very much for submitting this!

@drwl drwl merged commit 3d0e3f7 into drwl:main Jan 26, 2025
22 checks passed
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.

2 participants