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

Add support for job summaries πŸ“‘ #41

Merged
merged 1 commit into from
Jul 1, 2022

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Jun 29, 2022

Example of Differential ShellCheck job summary:

Differential ShellCheck 🐚

Changed scripts: 2

❌ Added βœ… Fixed
⚠️ Errors / Warnings / Notes 1 0

Useful links


ℹ️ When you have an issue with GitHub action please try to run it in debug mode and submit an issue.

@jamacku jamacku added the type: feature New feature label Jun 29, 2022
@jamacku jamacku self-assigned this Jun 29, 2022
@jamacku jamacku added this to the v2.2.0 milestone Jun 29, 2022
@jamacku jamacku linked an issue Jun 29, 2022 that may be closed by this pull request
@mergify mergify bot added the source label Jun 29, 2022
echo -e "\
### Differential ShellCheck 🐚

Changed scripts: \`${#list_of_changed_scripts[@]}\`

Check warning

Code scanning / csdiff

list_of_changed_scripts is referenced but not assigned.

list_of_changed_scripts is referenced but not assigned.
@jamacku jamacku force-pushed the add-summary branch 4 times, most recently from 4ffd37d to 59a54cc Compare June 29, 2022 09:42
@jamacku jamacku requested a review from lzaoral June 29, 2022 09:49
@jamacku jamacku changed the title Add support for job summaries Add support for job summaries πŸ“‘ Jun 30, 2022
@jamacku jamacku merged commit afe9df9 into redhat-plumbers-in-action:main Jul 1, 2022
@jamacku jamacku deleted the add-summary branch July 1, 2022 12:24

| | ❌ Added | βœ… Fixed |
|:---------------------------:|:-------:|:-------:|
| ⚠️ [Errors / Warnings / Notes](https://github.com/${GITHUB_REPOSITORY}/pull/${GITHUB_REF//merge}/files) | **${added_issues:-0}** | **${fixed_issues:-0}** |
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member Author

@jamacku jamacku Jul 7, 2022

Choose a reason for hiding this comment

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

Thank you! I came across this issue but a forgot about it 😺. This change should resolve the issue:

https://github.com/${GITHUB_REPOSITORY}/${${GITHUB_REF##refs\/}%%\/merge}/files

link -> https://github.com/redhat-plumbers-in-action/differential-shellcheck/pull/41/files

I'll submit PR.

jamacku added a commit that referenced this pull request Jul 7, 2022
jamacku added a commit that referenced this pull request Jul 7, 2022
jamacku added a commit that referenced this pull request Jul 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement support for GitHub Actions Job Summaries
2 participants