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

[CI] Updates GitHub Actions #1085

Draft
wants to merge 4 commits into
base: 7.x
Choose a base branch
from
Draft

[CI] Updates GitHub Actions #1085

wants to merge 4 commits into from

Conversation

picandocodigo
Copy link
Member

No description provided.

picandocodigo and others added 4 commits September 5, 2024 10:35
Lock bundler to v 2.4.22, since Ruby old :(
Removes unnecessary package installation
Updates STACK_VERSION
Adds jruby 9.4
Run bundle install with BUNDLE_GEMFILE
Updates actions/checkout
sqlite for Rails 5 shenanigans
ActiveSupport patches Kernel to add `class_eval` but this behavior wasn't
loaded in the test environment. This created a discrepancy between test
and prod, causing tests to fail that should have passed and vice versa.
Fully loading ActiveSupport makes the test environment more accurate.
The goal of ClassMethodsProxy is to avoid polluting the target's namespace,
but it was possible to do this by accident when calling `class_eval` before
ActiveSupport was completely loaded. This test ensures the namespace isn't
polluted, regardless of the load state of ActiveSupport.
@picandocodigo picandocodigo force-pushed the updates_7.x branch 2 times, most recently from cd57028 to 2939703 Compare September 5, 2024 09:39
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