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

Rails 6 suppport #63

Merged
merged 5 commits into from
Oct 2, 2023
Merged

Rails 6 suppport #63

merged 5 commits into from
Oct 2, 2023

Conversation

tf
Copy link
Member

@tf tf commented Oct 2, 2023

REDMINE-19438

Allow upgrading to Ruby 3. Only define lower compatibility bound from
now on to ease future updates

REDMINE-19438
Add paths to both `autoload_paths` and
`eager_load_paths`. `eager_load_paths` is required to have paths
covered by `zeitwerk:check`.

Ignore and require Paperclip processors and configuration, which are
cached during initialization and, thus, cannot be reloaded.

For Rails < 6, we cannot add `lib` to the `eager_load_paths` since
Resque eager loads [1], which then evaluates the engine class a second
time. This time `autoload_paths` is already frozen, resulting in an
error.

REDMINE-19438

[1] https://github.com/resque/resque/blob/v1.27.4/lib/resque/tasks.rb#L45
The reusable workflow now runs specs against experimental Rails
versions which require Pageflow to be installed from a git source.

REDMINE-19438
@tf tf added this to the v2.6 milestone Oct 2, 2023
@tf tf merged commit 0e05bb4 into codevise:master Oct 2, 2023
2 checks passed
@tf tf deleted the rails-6 branch October 2, 2023 08:06
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.

1 participant