foreman icon indicating copy to clipboard operation
foreman copied to clipboard

Fixes #33895 - Set up Zeitwerk inflector

Open ekohl opened this issue 1 year ago • 2 comments

Zeitwerk uses constant to snake case to resolve the paths. If we want to change some constants to be Uppercase, we need to use the inflector.

This is an alternative to https://github.com/theforeman/foreman/pull/8282. It may break plugins, so we'll need to verify those. It's also not complete to make Zeitwerk work.

ekohl avatar Mar 04 '24 13:03 ekohl

Katello does fail with this:

[2024-03-05T16:14:41.043Z] NameError: uninitialized constant RepoRenameFeedToURL
[2024-03-05T16:14:41.043Z] Did you mean?  RepoRenameFeedToUrl

Which makes me wonder: will we be able to keep it compatible or do all plugins need to bump?

ekohl avatar Mar 05 '24 19:03 ekohl

Reflecting what @ofedoren and I discussed: he'll look at an alternative implementation that's less invasive.

ekohl avatar Mar 27 '24 12:03 ekohl

Will be continued in https://github.com/theforeman/foreman/pull/10131.

ekohl avatar Jun 18 '24 15:06 ekohl