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

status-update: Don't comment when there are pushes #139

Merged
merged 1 commit into from
Jun 15, 2016

Conversation

emilio
Copy link
Member

@emilio emilio commented Jun 14, 2016

GitHub has started to send emails when commits are pushed.

If there's any other reason for keeping it, happy to close it :)

@highfive
Copy link
Collaborator

Heads up! This PR modifies the following files:

  • @jdm: handlers/status_update/init.py

@highfive
Copy link
Collaborator

New code was committed to pull request.

@jdm
Copy link
Member

jdm commented Jun 15, 2016

Flake8 is displeased, but otherwise this is fine to merge.

GitHub has started to send emails when commits are pushed.

If there's any other reason for keeping it, happy to close it :)
@highfive
Copy link
Collaborator

New code was committed to pull request.

@emilio
Copy link
Member Author

emilio commented Jun 15, 2016

Added the missing newline.

@emilio emilio merged commit d4ca52f into servo:master Jun 15, 2016
@emilio emilio deleted the push-notification branch June 15, 2016 00:43
Mark-Simulacrum pushed a commit to Mark-Simulacrum/highfive that referenced this pull request Sep 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants