-
Notifications
You must be signed in to change notification settings - Fork 407
Add ability to archive a repository #758
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
Conversation
Note: I'm working on re-recording these cassettes. |
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.
This looks good, but did you want to add a couple tests to cover it?
Uh, sorry, the tests are failing horribly on this because we need to re-record the cassettes. Looks like accidentally deleting this removed AppVeyor and Travis. Let me add those back. |
Finished re-recording the universe. Hopefully now these test will pass :) |
Ok @omgjlk I added a commit with an explicit test that this works. |
Once #770 merges, I'm going to close and re-open this to trigger a new merged build. |
Ah, the push CI will fail because of this... Let me rebase. sigh |
Also add the attribute to determine whether a given Repository is archived or not. While we're here, let's also trim out some dead attributes that seem to have come from our PullRequest objects. Closes gh-752
Remove chitauri-preview header and use SHA. This was found while re-recording cassettes and wasn't easy to factor out into its own PR. While fixing the tests, I also found we had tests that were magically passing earlier (regarding Repository.{source,parent}) so that's fixed now.
7375fbf
to
3af6035
Compare
Also add the attribute to determine whether a given Repository is
archived or not. While we're here, let's also trim out some dead
attributes that seem to have come from our PullRequest objects.
Closes gh-752