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

Support commenting on PRs #17

Merged
merged 4 commits into from
Oct 27, 2016
Merged

Support commenting on PRs #17

merged 4 commits into from
Oct 27, 2016

Conversation

orta
Copy link
Member

@orta orta commented Oct 27, 2016

Still not really doing much testing yet, I'm starting to feel close on having all the structures in place around Danger JS so that I can start writing tests.

This will also remove the fail on non-PR builds.

Hopefully, she should respond about the lack of a changelog entry.

@orta orta force-pushed the comment branch 8 times, most recently from 9ad784f to c361423 Compare October 27, 2016 11:55
@DangerCI
Copy link

No Changelog changes!

@orta orta merged commit cd53d8f into master Oct 27, 2016
@orta orta deleted the comment branch October 27, 2016 12:07
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.

2 participants