Skip to content

to more rails 7.1 defaults #1239

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.

Sign up for GitHub

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

Jump to bottom
Draft
wwahammy wants to merge 22 commits into
base: to-rails-7.1-defaults
Choose a base branch
from
Draft

to more rails 7.1 defaults #1239

wwahammy wants to merge 22 commits into from

Conversation

@wwahammy
Copy link
Member

@wwahammy wwahammy commented Jun 28, 2025

  • Turn on config.add_autoload_paths_to_load_path = false
  • Fix zeitwerk error for Mailchimp
  • Improve to_deprecated_h initializer
  • Remove autoloaded format/name require
  • Remove unneeded requires from HealthReport
  • Remove unused requires in InsertActivities
  • Remove unused requires
  • Remove unused requires
  • Delete more requires
  • Remove unused requires
  • Remove more requires
  • Remove the unneed autoloads

NOTE: DO NOT discuss internal CommitChange information in your PR; this PR will be public.
Link back to the issue in the Tix repo when you need to do that.

wwahammy added 22 commits June 26, 2025 11:22
@wwahammy
Turn on config.add_autoload_paths_to_load_path = false
0024b23
@wwahammy
Fix zeitwerk error for Mailchimp
30436f1
@wwahammy
Improve to_deprecated_h initializer
7862a49
@wwahammy
Remove autoloaded format/name require
7f3a8f0
@wwahammy
Merge branch 'remove-unused-nonprofits_controller#create' into to-rai…
06c8faf 
…ls-7.1-defaults
@wwahammy
Remove unneeded requires from HealthReport
289e2d6
@wwahammy
Remove unused requires in InsertActivities
2614b69
@wwahammy
Remove unused requires
95908c3
@wwahammy
Remove unused requires
efdabaf
@wwahammy
Delete more requires
4cd6b8c
@wwahammy
Merge branch 'remove-empty-class' into to-rails-7.1-defaults
5cc725c
@wwahammy
Remove unused requires
b024423
@wwahammy
Remove more requires
0160697
@wwahammy
Remove the unneed autoloads
afc316a
@wwahammy
Merge branch 'to-rails-7.1-defaults' into to-more-rails-7.1-defaults
5f9c2ac
@wwahammy
Merge branch 'to-rails-7.1-defaults' into to-more-rails-7.1-defaults
56e7e92
@wwahammy
Remove unused require "cypher" from InsertNonprofitKeys
0981c2d
@wwahammy
Remove unused Hash.with_defaults_unless_nil method
e00500a
@wwahammy
Merge branch 'remove-unused-hash' into to-more-rails-7.1-defaults
57c84a7
@wwahammy
Merge branch 'remove-unused-require' into to-more-rails-7.1-defaults
11946a6
@wwahammy
Undo the encryption change
e531656
@wwahammy
Set active_support.raise_on_invalid_cache_expiration_time = true be…
968183c 
…cause I've looked at every use of Rails.cache and they all seem to be done in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@wwahammy