-
Notifications
You must be signed in to change notification settings - Fork 347
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
Job summary on PR #1058
Comments
I understand that you are asking for a feature enhancement to the action. Could you explain a little more what you would like and how it would work? Please also describe why this is needed and what benefit it would give. |
Many thanks for the very quick response. The general idea is to have an option to publish the test summary to the respective PR as a comment. If the parameter The benefit of said feature is to deliver the option to centralize relevant info on the PR and avoid having to go to the Actions tab. Many thanks |
It's an interesting idea. I guess we have to wait to see what the Cypress team thinks and if anybody is prepared to submit a PR to implement it. The best alternative at the moment is to view the results in the PR, for example #1059 which I submitted today shows: however clicking on any of the relevant Details link like https://github.com/cypress-io/github-action/actions/runs/6559572703/job/17815434926?pr=1059 does not show the Job Summary Only if you click on the Summary over on the left do you then get to the desired overview page |
Exactly, my idea is to avoid having to navigate to the Action summary. |
Would love to see this too |
I also wish that this action could add the summary into the PR, that is exactly what Codecov is doing and it is super convenient to get all the info directly in the PR (see below). Another approach could be to use this other GitHub Action EnricoMi/publish-unit-test-result-action that collects any test reports and displays the summary into multiple places (as a PR comment, as a Job Summary, ...) but that requires us to run Cypress with reporters as JUnit or JSON which is then passed to that At the end of the day, Cypress already have all the info displayed in the Job Summary, it's just a matter of sending it to the PR as well. Note that it might need extra permissions to write to the PR but that's just a setup that user could accept to opt-in or not (e.g. |
Honestly just adding the grid to the output would be a huge boon, then people could do the commenting bit themselves. |
If you could have the job summaries of all the jobs displayed at the bottom of the PR, that could negate the need for having to comment at all. My issue at the moment is that we have seperate workflows that are trying to publish information to the PR but we end up with comment spam because, whilst they each use a comment maintain action, theres still 5 comments for each PR. Job summaries would be good, but with the details link going to the job page rather than summary, i have to guide the devs to the output data. |
Got here from Google search because we are also looking for the exact same feature (which in the end sounds like a GitHub feature request instead):
I'm also not sure why the link sends to the logs/details instead of the job summary... The summary would be a much more helpful starting point (and from there we can dig into the details if necessary). |
Could have an option to publish the job summary to the PR please.
The text was updated successfully, but these errors were encountered: