-
Notifications
You must be signed in to change notification settings - Fork 357
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
Menu Simplification: Drop the Optimize section #5485
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.
Tested. Looks fine.
Travis failure is legit! |
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.
Fix the specs.
Checked commit skateman@21c8f86 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 spec/presenters/tree_builder_ops_rbac_features_spec.rb
|
@miq-bot add_label changelog/yes |
As
Utilization
went into theCloud Intel
and the remaining two items underOptimize
will also go away, I am dropping the menu section completely. Sorry @GregP for the caused merge conflicts, we got these colliding RFEs, but it's just deletion, so probably easy to rebase.Before:
After:
@miq-bot add_reviewer @epwinchell
@miq-bot add_label ux/review, hammer/no
https://bugzilla.redhat.com/show_bug.cgi?id=1678196