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

Fix Rails 7.2 Deprecation warnings #19678

Merged
merged 5 commits into from
Dec 4, 2024
Merged

Conversation

ryan-mcneil
Copy link
Contributor

@ryan-mcneil ryan-mcneil commented Dec 2, 2024

Summary

  • Fix Deprecation warnings blocking Rails 7.2+ upgrade

Notes

  • DEPRECATION WARNING: config.active_support.remove_deprecated_time_with_zone_name is deprecated and will be removed in Rails 7.2.
    • Fixed by removing the deprecated config option config.active_support.remove_deprecated_time_with_zone_name. We had it set to false, suggesting we used the old standard, the more "human readable" format instead of the new standard. For example American/New_York vs the old Eastern Time (US & Canada). As this didn't break any specs, I think it's ok.
  • DEPRECATION WARNING: Your secret_key_baseis configured inRails.application.secrets, which is deprecated in favor of Rails.application.credentials and will be removed in Rails 7.2.
    • We don't use secrets.yml (or the newer credentials.yml.enc for that matter).
    • No issues removing secrets.yml, as there are no references to Rails.application.secrets

Related issue(s)

Testing done

  • Specs Passing and local server booting after making changes

Copy link

github-actions bot commented Dec 2, 2024

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: lib/config_helper.rb

@jla3378

This comment was marked as spam.

@va-vfs-bot va-vfs-bot temporarily deployed to rm-fix-7.2-deprecation-warnings/main/main December 2, 2024 22:54 Inactive
Copy link

github-actions bot commented Dec 4, 2024

Error: A file (or its parent directories) does not have a CODEOWNERS entry. Please update the .github/CODEOWNERS file and add the entry for the Offending file: lib/config_helper.rb

@va-vfs-bot va-vfs-bot temporarily deployed to rm-fix-7.2-deprecation-warnings/main/main December 4, 2024 00:12 Inactive
Copy link

github-actions bot commented Dec 4, 2024

Error: A file (or its parent directories) was deleted but its reference still exists in CODEOWNERS. Please update the .github/CODEOWNERS file and delete the entry for the Offending file: config/secrets.yml

@ryan-mcneil ryan-mcneil merged commit 1303e96 into master Dec 4, 2024
25 checks passed
@ryan-mcneil ryan-mcneil deleted the rm-fix-7.2-deprecation-warnings branch December 4, 2024 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants