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

chore: bump deprecated actions #100

Merged
merged 1 commit into from
Nov 13, 2024
Merged

Conversation

simonpasquier
Copy link
Contributor

No description provided.

@simonpasquier simonpasquier requested a review from a team as a code owner November 7, 2024 10:15
@simonpasquier simonpasquier force-pushed the bump-actions branch 7 times, most recently from 2ea89a9 to d652a89 Compare November 7, 2024 10:44
@slashpai
Copy link
Member

github action review shows a few errors

@rexagod
Copy link
Member

rexagod commented Nov 11, 2024

Should we consider dropping actionlint in favor of a workflow (that also actively blocks PRs) since the current errors don't seem pinpoint the issues they indicate correctly? I believe atleast one of these were carried-over from previous patches that went unnoticed.

@simonpasquier simonpasquier force-pushed the bump-actions branch 2 times, most recently from 3702fe3 to 7d5b014 Compare November 12, 2024 15:31
@@ -6,3 +6,7 @@ jobs:
steps:
- uses: actions/checkout@v4
- uses: reviewdog/action-actionlint@v1
with:
fail_on_error: "true"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rexagod @slashpai PRs should now fail the CI if actionlint isn't happy.

Signed-off-by: Simon Pasquier <spasquie@redhat.com>
@simonpasquier simonpasquier merged commit 05f1c77 into rhobs:main Nov 13, 2024
9 checks passed
@simonpasquier simonpasquier deleted the bump-actions branch November 14, 2024 07:59
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.

3 participants