Skip to content

feat: add summary to action output#659

Open
R-Raposo wants to merge 1 commit intodorny:mainfrom
swisspost:main
Open

feat: add summary to action output#659
R-Raposo wants to merge 1 commit intodorny:mainfrom
swisspost:main

Conversation

@R-Raposo
Copy link

This change implements adding the summary to the action's output, as discussed here.

There's this PR open #607, but I think it doesn't quite work according to tests I did, with workflows using this action with the changes applied.

@abelbraaksma
Copy link
Contributor

This is great and would open the door for writing test summary comments! Hope @dorny can take a look? 🙏

@jozefizso
Copy link
Collaborator

Hi @R-Raposo, thanks for this change.

Please, keep the PR focused to only adding the summary output variable.

@jozefizso jozefizso self-requested a review December 29, 2025 14:38
Copy link
Collaborator

@jozefizso jozefizso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove the changes to dotnet-trx and golang-json parsers as those are unrelated to this PR.

@jozefizso jozefizso added the enhancement New feature or request label Dec 29, 2025
@jozefizso
Copy link
Collaborator

Plase, do changes on feature branches, as PRs submitted from the main branch are to editable by maintainers.

@R-Raposo R-Raposo force-pushed the main branch 2 times, most recently from bde759b to 585faac Compare January 12, 2026 11:47
@R-Raposo
Copy link
Author

R-Raposo commented Jan 12, 2026

hello @jozefizso, can you please check the PR again, to see if it's ok now? I've got some changes to the licences file when running npm run package, let me know if I should remove this from the PR.

@R-Raposo
Copy link
Author

Plase, do changes on feature branches, as PRs submitted from the main branch are to editable by maintainers.

Sorry, we are already using a fork of this repo in SwissPost, so it was more convenient to just create a PR from the fork. Will keep in mind for the future.

@R-Raposo R-Raposo requested a review from jozefizso January 12, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants