Actions
Refactor #20611
closedRemove legacy config code
Difficulty:
easy
Triaged:
Pull request:
Team Backlog:
Description
Hammer still contains legacy code that handles:
- Loading config files from ~/.foreman and /etc/foreman (only ~/.hammer and /etc/hammer are supported now)
https://github.com/theforeman/hammer-cli/blob/master/bin/hammer#L55-L63
- Loading cli modules config files from $CONF_DIR/hammer.modules.d/ (only $CONF_DIR/cli.modules.d/ is supported now)
https://github.com/theforeman/hammer-cli/blob/master/lib/hammer_cli/settings.rb#L26-L29
- Enabling plugins via list of their names in :modules: in the settings (each plugin now defines its own section in the config and uses :enable_module: true/false)
https://github.com/theforeman/hammer-cli/blob/master/lib/hammer_cli/modules.rb#L10-L24
The deprecation warnings have been there for quite some time. It's safe to remove the legacy code now.
Updated by The Foreman Bot over 7 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/hammer-cli/pull/255 added
Updated by Oleh Fedorenko over 7 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset ef9ac981e54fbc5fd716b4b4243265bc980296c8.
Actions