Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Fix bug with closing account w/ closed project #4289

Merged
merged 2 commits into from
Jan 12, 2017
Merged

Conversation

chadwhitacre
Copy link
Contributor

Part of #3602, should've happened in #4287.

@chadwhitacre chadwhitacre added this to the Finish Project Basics milestone Jan 12, 2017
@chadwhitacre
Copy link
Contributor Author

Ready for review, @mattbk, et al. Should fix #3602 (comment).

@chadwhitacre
Copy link
Contributor Author

The second build for the first commit pulled the second commit, so it passed when it shouldn’t have.

@chadwhitacre
Copy link
Contributor Author

I can repush but you can probably get the picture with the first build on the commit. ;-)

@chadwhitacre
Copy link
Contributor Author

The push build on the second commit also failed, but that was due to a spurious test failure (#4290). Restarted ...

@mattbk
Copy link
Contributor

mattbk commented Jan 12, 2017

  • Closed a team
  • Closed the owner account

Success!

@chadwhitacre
Copy link
Contributor Author

Huzzah! Now we just need Travis ...

@mattbk mattbk merged commit 8d10188 into master Jan 12, 2017
@mattbk mattbk deleted the close-with-closed-team branch January 12, 2017 22:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants