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 #13614 - Make capsule only an alias to proxy #349
Conversation
3216416
to
c311a27
Compare
c311a27
to
493163d
Compare
@@ -24,6 +24,8 @@ def self.exception_handler_class | |||
require 'hammer_cli_katello/id_resolver' | |||
|
|||
# commands | |||
require 'hammer_cli_katello/capsule' |
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.
Why can't this be lumped into the list of other requires just a few lines up?
1ddff99
to
0ba5dbe
Compare
@komidore64 thanks, updated. |
@@ -136,7 +108,9 @@ class SyncCommand < HammerCLIForemanTasks::AsyncCommand | |||
autoload_subcommands | |||
end | |||
|
|||
autoload_subcommands | |||
HammerCLIForeman::SmartProxy.subcommand("content", |
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.
One more thing. Could we use HammerCLIKatello::Capsule::Content.command_name
here rather than "content"
? Same reason that I asked about using #desc()
, it's one place to keep track of the command name, rather than two.
ACK pending comment. |
0ba5dbe
to
d99c390
Compare
Fixed |
@tstrachota mind rebasing? Also, is this intended for 3.0? We branched yesterday and this will need a BZ attached to it, if you want it cherry-picked into 3.0. |
d99c390
to
2ba9e4e
Compare
@komidore64 I rebased, moved the issue to Katello and cloned it to bugzilla. |
@tstrachota sweet! 🐈 |
Fixes #13614 - Make capsule only an alias to proxy
No description provided.