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
Comment on the pull request #187
Comments
Can this be achieved with #178? |
I personally think providing the error as an Action Step Output is different to enabling the Action to comment the issue and a suggestion on the PR itself. I would suggest these are separate issues, though my comment in #178 was due to the OPs Issue body seems to indicate wanting PR comments vs the Issue title requests Step output for manual composition in Actions. I would love to see some terse error with a suggestion, like https://github.com/jef/conventional-commits-pr-action but with less text. |
People can definitely build their own logic to add comments using #178. But that's pretty cumbersome. I think notifying the user about incorrect formatting of their pull request is a part of the core feature set of this GitHub Action. |
As I state above, I think commenting on PRs and Action outputs are different. Can we reopen this or should I create another Issue to specifically track comments on PR feature request? |
To be honest I currently don't really see the value of commenting on pull requests. I'd like to keep the noise of this action to a minimum, so in case you follow the provided configuration, you shouldn't hear anything from this action. Only when things go wrong there should be an error message and I think checks work really well for that. Having a comment repeat what a check already provides doesn't seem useful to me. However, with #178 you could build such a workflow yourself if you have a use case where this makes more sense. I saw there are actions like https://github.com/marketplace/actions/comment-pull-request which might help to redirect the output to a PR comment. |
I disagree. Seeing that the check failed does not tell you why it failed. You have to click through via the "details" link to see the Action execution details. From my experience on Not to mention this could be opt-in, so it's only noisy for those who want the noise. I want the noise to stop me having to make noise manually. |
At our project, many non-developers use GitHub pull requests to manage things like blog posts or job postings. Most of these users are not comfortable clicking on a broken CI status and sifting through very technical steps and their console output. I think getting a friendly comment on a pull request that explains in plain English (or French|Spanish|Chinese) what exactly the problem is and links to further documentation is a lot more user friendly for these people than the current behavior. I think it would be completely okay if this feature is disabled by default. This GitHub Action doesn't even need to provide a default text. All that is needed from my perspective is a string input parameter that when set posts the given text as a comment on the pull request. |
It would be nice if this action would also add a comment to the pull request directly. Here is an example for how that could look like:
The message should - of course - be configurable via an input option.
Implementing this is pretty straightforward:
The text was updated successfully, but these errors were encountered: