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 GitHub Action examples #1135

Merged
merged 3 commits into from Jun 19, 2022
Merged

Update GitHub Action examples #1135

merged 3 commits into from Jun 19, 2022

Conversation

abitrolly
Copy link
Contributor

@abitrolly abitrolly commented Jun 13, 2022

  • updates versions (may be more performant)
  • added with: into text for copy and documented config-file
  • explain download unpacking files into dist/

@abitrolly abitrolly changed the title Update to latest GitHub Action versions Update GitHub Action examples Jun 13, 2022
docs/setup.md Outdated Show resolved Hide resolved
docs/setup.md Outdated Show resolved Hide resolved
docs/setup.md Outdated Show resolved Hide resolved
  checkout@v2 -> v3
  upload/download-artifact@v2 -> v3
  python-setup@v2 -> v3
  pypa/gh-action-pypi-publish@1.4.2 -> v1.5.0

Python latest is v4, but it breaks working with default Python version
actions/setup-python#421
@abitrolly abitrolly force-pushed the patch-1 branch 2 times, most recently from d6c9b3d to 106b399 Compare June 18, 2022 12:24
@abitrolly
Copy link
Contributor Author

This is ready to review now.

Copy link
Contributor

@henryiii henryiii left a comment

Choose a reason for hiding this comment

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

Looks fine to me.

Copy link
Contributor

@joerick joerick left a comment

Choose a reason for hiding this comment

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

Looks good. Thank you!

@joerick joerick merged commit 0bf4f7a into pypa:main Jun 19, 2022
@abitrolly abitrolly deleted the patch-1 branch June 19, 2022 09:14
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.

None yet

4 participants