-
Notifications
You must be signed in to change notification settings - Fork 4
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
Github team_member? method #113
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.
@LanceHolsten, the code in here looks good. Please add a test similar to one of the other ones in:
https://github.com/sportngin/octopolo/blob/master/spec/octopolo/github_spec.rb
I realize my two tests test the exact same flow, but I wrote two (one for each possible outcome) in case it's important to illustrate all possibilities. Note: Also I wouldn't be surprised if this isn't how these are meant to be tested... |
|
What
Adding a
team_membership?
method to the Github module in order to apply restrictions on certain AutoMobile commandsWhy
To restrict the use of new AutoMobile commands to certain users
Rollback Plan
git revert -m 1 MERGE_SHA
and perform another deploy.URLs
QA Plan
I have tested this change in conjunction with AutoMobile's #154 using the auto_mobile_test repo with success, but I am not sure how to test this method while stand-alone.