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

CI: updated all deprecated actions #400

Merged
merged 2 commits into from
Jul 24, 2024

Conversation

Bullrich
Copy link
Contributor

I have noticed there are a lot of outdated actions with deprecated warnings:
image

This PR updates all of GitHub actions to latest version to remove such warnings.

I have also added someTODO to actions that have been archived by GitHub and that require an alternative action/solution. We can do that in a different PR once we identify the correct solution.

  • Does not require a CHANGELOG entry

@@ -150,6 +150,7 @@ jobs:

- name: Create release
id: create-release
# TODO: Replace as it has been deprecated
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems that this should be replaced by gh cli.

Example solution:

      - name: Create release
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
          tag: ${{ github.ref_name }}
        run: |
          gh release create "$tag" \
              --repo="$GITHUB_REPOSITORY" \
              --title="${GITHUB_REPOSITORY#*/} ${tag#v}" \
              --generate-notes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

GitHub also showed some alternatives actions in the readme

Comment on lines +187 to 189
# TODO: Replace as it has been deprecated
uses: actions/upload-release-asset@v1
env:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

From the action's readme the new action that seems to replace this one is softprops/action-gh-release

@Bullrich
Copy link
Contributor Author

/merge

Copy link

Review required! Latest push from author must always be reviewed

@bkchr
Copy link
Contributor

bkchr commented Jul 24, 2024

/merge

@fellowship-merge-bot fellowship-merge-bot bot merged commit 4eb5267 into polkadot-fellows:main Jul 24, 2024
44 of 45 checks passed
@fellowship-merge-bot
Copy link
Contributor

Enabled auto-merge in Pull Request

Available commands
  • /merge: Enables auto-merge for Pull Request
  • /merge cancel: Cancels auto-merge for Pull Request
  • /merge help: Shows this menu

For more information see the documentation

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