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

Update to Scala Steward GHA version that fixes the run-summary #78

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

rtyley
Copy link
Member

@rtyley rtyley commented Nov 1, 2024

This will restore the Scala Steward GitHub Job run-summary that went missing a few days ago.

This is what the run-summary normally looks like - a report on each individual repo the Scala Steward job ran on:

Boo

When I merged #75 on 30th October, the version of the Scala Steward GHA was updated to include a regression that lost the run summary - when I noticed this, I submitted a PR to fix the bug, which has since been merged & released with v2.71.0:

@rtyley rtyley force-pushed the use-fixed-scala-steward-action-with-pr-652 branch from 085ba4c to cc46545 Compare November 5, 2024 10:38
@rtyley rtyley marked this pull request as ready for review November 5, 2024 10:51
@rtyley rtyley changed the title Try out scala-steward-action PR 652 Update to Scala Steward GHA version that fixes run summary Nov 5, 2024
@rtyley rtyley requested a review from a team November 5, 2024 11:36
@rtyley rtyley changed the title Update to Scala Steward GHA version that fixes run summary Update to Scala Steward GHA version that fixes the run-summary Nov 5, 2024
Copy link
Member

@kelvin-chappell kelvin-chappell left a comment

Choose a reason for hiding this comment

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

👍

I wonder why dependabot didn't pick up the new release.
I suppose the release was too recent.

@rtyley rtyley merged commit ea88a88 into main Nov 5, 2024
@rtyley rtyley deleted the use-fixed-scala-steward-action-with-pr-652 branch November 5, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants