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

Add Configurable Inflector for Zeitwerk #796

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

jclusso
Copy link
Contributor

@jclusso jclusso commented Aug 22, 2023

This is a 🙋 feature or enhancement.
This is a 🔦 documentation change.

Summary
Since #793 is a breaking change I've introduced this PR that allows you to configure an inflector. The documentation is updated to show you how to configure ActiveSupport::Inflector and notes that it will become the default in v2.0.

Disable RuboCop Metrics/BlockLength

Add ActiveSupport::Inflector to Zeitwerk

- Configure Zeitwerk's inflector to use ActiveSupport::Inflector
- Add commented out example to config/initializer.rb
- Update existing documentation about ActiveSupport::Inflector to reference initializer configuration
- Add documentation for initializer
@jaredcwhite
Copy link
Member

This is awesome. Thanks @jclusso!

@jaredcwhite jaredcwhite merged commit ecdff83 into bridgetownrb:main Aug 29, 2023
4 checks passed
@jclusso jclusso deleted the add_configurable_inflector branch August 29, 2023 19:37
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