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 many of the actions to their latest version #1731

Merged
merged 10 commits into from
Apr 28, 2023

Conversation

Panquesito7
Copy link
Member

@Panquesito7 Panquesito7 commented Apr 12, 2023

Things added/changed:

  • Update many of the actions to their latest version.

@Panquesito7 Panquesito7 added the devops Use this label for devops related enhancements label Apr 12, 2023
.github/workflows/codeql-analysis.yml Outdated Show resolved Hide resolved
.github/drafts/optimize_svg.yml Outdated Show resolved Hide resolved
@@ -5,7 +5,7 @@ on:

jobs:
stale:
runs-on: ubuntu-latest
runs-on: ubuntu-latestlatest
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
runs-on: ubuntu-latestlatest
runs-on: ubuntu-latest

.github/workflows/peek_icons.yml Outdated Show resolved Hide resolved
@Panquesito7
Copy link
Member Author

I've tested a few workflows on my fork and they seem to be working fine.
Please check again and make sure everything's correct. Thanks. 🙂

Co-authored-by: Jørgen Kalsnes Hagen <43886029+Snailedlt@users.noreply.github.com>
Copy link
Collaborator

@Snailedlt Snailedlt left a comment

Choose a reason for hiding this comment

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

Statically it looks good, and I trust you tested all of the workflows properly :)

Thanks for the contributions @Panquesito7 🙇

@Panquesito7
Copy link
Member Author

Statically it looks good, and I trust you tested all of the workflows properly :)

Thank you for reviewing! In case there's anything wrong, we can always revert this PR. 😉

Copy link
Contributor

@lunatic-fox lunatic-fox left a comment

Choose a reason for hiding this comment

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

Great work! 🚀
Just one suggestion about dawidd6/action-download-artifact action version and @Snailedlt suggestion on .github/drafts/stale.yml.

@@ -20,22 +20,22 @@ jobs:
run: echo $PEEK_STATUS

- name: Download workflow artifact
uses: dawidd6/action-download-artifact@v2.11.0
uses: dawidd6/action-download-artifact@v2.26.1
Copy link
Contributor

Choose a reason for hiding this comment

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

There is a recently update to that action.

Suggested change
uses: dawidd6/action-download-artifact@v2.26.1
uses: dawidd6/action-download-artifact@v2.27.0

steps:
- name: Check if the trigger run worked. If it failed, fail the current run.
if: github.event.workflow_run.conclusion != 'success'
uses: cutenode/action-always-fail@v1.0.1

- name: Download workflow artifact
uses: dawidd6/action-download-artifact@v2.11.0
uses: dawidd6/action-download-artifact@v2.26.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
uses: dawidd6/action-download-artifact@v2.26.1
uses: dawidd6/action-download-artifact@v2.27.0

@Panquesito7 Panquesito7 merged commit 2b9faad into develop Apr 28, 2023
@Panquesito7
Copy link
Member Author

Weird, GitHub did not notify me about the reviews 👀 let me update those. Thanks, @lunatic-fox. 🙂

@Panquesito7 Panquesito7 deleted the panquesito7/update/actions branch April 28, 2023 16:22
@@ -7,7 +7,7 @@ on:
jobs:
post_screenshots_in_comment:
name: Post the screenshot
runs-on: ubuntu-18.04
runs-on: ubuntu-latest
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠ This change will take effect when this commit is merged to master, as for now this action will be loading indefinitely to find a ubuntu-18.04 machine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops Use this label for devops related enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants