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

Revert "chirp: 20161018 -> 20170311" #24095

Merged
merged 1 commit into from Mar 20, 2017
Merged

Revert "chirp: 20161018 -> 20170311" #24095

merged 1 commit into from Mar 20, 2017

Conversation

ndowens
Copy link
Contributor

@ndowens ndowens commented Mar 20, 2017

Reverts #24094

@mention-bot
Copy link

@ndowens, thanks for your PR! By analyzing the history of the files in this pull request, we identified @the-kenny, @zimbatm and @fpletz to be potential reviewers.

@ndowens ndowens merged commit 18cf033 into master Mar 20, 2017
@ndowens ndowens deleted the revert-24094-chirp branch March 20, 2017 02:41
@fpletz
Copy link
Member

fpletz commented Mar 20, 2017

Why? What are you doing?!

@ndowens
Copy link
Contributor Author

ndowens commented Mar 20, 2017 via email

@fpletz
Copy link
Member

fpletz commented Mar 20, 2017

What?

@ndowens
Copy link
Contributor Author

ndowens commented Mar 20, 2017 via email

@fpletz
Copy link
Member

fpletz commented Mar 20, 2017

So why revert the commit and push it again? Do you know how to use git?

@the-kenny
Copy link
Contributor

Also, is there any reason for the immediately-merged PRs? You can just push your commits directly to master and prevent a lot of noise (in the commits as well as in other contributor's emails)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants