Skip to content
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

[EUWE] Improve rbac tree spec #14445

Merged
merged 3 commits into from
Apr 3, 2017
Merged

Conversation

hayesr
Copy link
Contributor

@hayesr hayesr commented Mar 21, 2017

This change minimizes the features loaded before each example in this spec. Follow-up to #14431

/cc @kbrock @jrafanie @Fryguy

@miq-bot miq-bot changed the title Improve rbac tree spec [EUWE] Improve rbac tree spec Mar 21, 2017
@jrafanie
Copy link
Member

Yay, net deletion too! +5 −8

✂️ 🔥 ✂️ 🔥 ✂️ 🔥

🍰 🍪 👏 🙇 😍 🎉

@Fryguy Fryguy assigned simaishi and unassigned Fryguy Mar 22, 2017
@kbrock
Copy link
Member

kbrock commented Mar 27, 2017

These changes look good.
kicking to try and get this green

@kbrock kbrock closed this Mar 27, 2017
@kbrock kbrock reopened this Mar 27, 2017
Copy link
Member

@kbrock kbrock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Eric,

This looks good

@kbrock
Copy link
Member

kbrock commented Mar 29, 2017

Travis, y u no green?

@kbrock kbrock closed this Mar 29, 2017
@kbrock kbrock reopened this Mar 29, 2017
@miq-bot
Copy link
Member

miq-bot commented Mar 30, 2017

Checked commits hayesr/manageiq@9b4dad1~...4277026 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 0 offenses detected
Everything looks good. 🏆

@hayesr
Copy link
Contributor Author

hayesr commented Mar 30, 2017

@kbrock @jrafanie Green again after #14557!

@jrafanie
Copy link
Member

jrafanie commented Apr 3, 2017

Yay! LGTM @hayesr. Only @simaishi merges backports so I'll let her merge when it's the right time.

@simaishi simaishi merged commit 7178b03 into ManageIQ:euwe Apr 3, 2017
@simaishi simaishi added this to the Sprint 58 Ending Apr 10, 2017 milestone Apr 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants