Y
Hacker News
new
|
ask
|
show
|
jobs
by
nl5887
4153 days ago
This is exactly why we've develop this. To enforce code quality to pull requests and save the repository owners time.
1 comments
bhuga
4153 days ago
I think he's commenting on the approach of how the system reports, not its usefulness. Have you considered bundling up the results as a second commit status[1] rather than making comments?
1:
https://github.com/blog/1935-see-results-from-all-pull-reque...
link
jbrooksuk
4153 days ago
Adding a commit status would be nice. It'd fail whilst the most recent commit has had a commit-comment on it.
link
nl5887
4153 days ago
The commit status have been added :-).
link
bhuga
4153 days ago
That was quick. Awesome! Do you have a sample pull with the new setup?
link
1: https://github.com/blog/1935-see-results-from-all-pull-reque...