-
Notifications
You must be signed in to change notification settings - Fork 99
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
Fixes #37452 - Support Zeitwerk loader #752
Fixes #37452 - Support Zeitwerk loader #752
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we make ID
an acronym to avoid renaming?
We could, but it will imply that we would need to either make custom inflector for the plugin, which won't affect core and other plugins or make it acronym in the core, so we have consistency. The later will bring even more changes across the plugins :/ |
4b6401a
to
2b4b7e7
Compare
2b4b7e7
to
dd3cb93
Compare
dd3cb93
to
6f742d6
Compare
6f742d6
to
35bfeab
Compare
e1d2cbc
to
35fa271
Compare
Ready to merge after theforeman/foreman#10131 is merged, dropped the commit for testing with Foreman PR, the latest run was green: |
35fa271
to
c0e6bc4
Compare
Thank you @ofedoren ! |
WIP