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

Upgrade Rails to 7.2 #1303

Open
wants to merge 24 commits into
base: develop
Choose a base branch
from
Open

Upgrade Rails to 7.2 #1303

wants to merge 24 commits into from

Conversation

caitmich
Copy link
Contributor

@caitmich caitmich commented Oct 3, 2024

Summary

Upgrade Rails to 7.2:

  • Use named arguments in serialize: calls
  • Wrap active_record.yaml_column_permitted_classes in an after_initialize block
  • Update config.action_dispatch.show_exceptions to use new :none value

I assign all rights, including copyright, to any future Dradis
work by myself to Security Roots.

Check List

  • Added a CHANGELOG entry

…t psych can allow the correct classes when using safe_load
… and define the list_item_name association in acts_as_linked_list. Add comment for clarity
…nfig.fixture_path to use new plural form & array value
…ed on the real models. Now that we're defining a belongs_to association in `acts_as_linked_list`, we need to test this on a real AR model
…ntroller`.

Rename `JSONWithIndifferentAccess` to use Json instead
…tements in node properties, add additional dirs to autoload_lib ignore list
bin/setup Outdated Show resolved Hide resolved
config/application.rb Outdated Show resolved Hide resolved
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