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: update action versions #267

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

mayeut
Copy link
Contributor

@mayeut mayeut commented May 4, 2024

This removes Node.js 16 deprecation warnings.

@mayeut mayeut requested a review from a team as a code owner May 4, 2024 13:30
@mayeut mayeut marked this pull request as draft May 4, 2024 13:36
@mayeut mayeut marked this pull request as ready for review May 4, 2024 13:41
@mayeut mayeut force-pushed the update-actions branch 2 times, most recently from 3fae550 to 8e3775e Compare May 9, 2024 14:19
This removes Node.js 16 deprecation warnings
@mayeut
Copy link
Contributor Author

mayeut commented May 10, 2024

same weird failure as in #273, ubuntu 24.04 tests were not executed on the correct image: https://github.com/actions/python-versions/actions/runs/9035848592/job/24831831531#step:1:5

@HarithaVattikuti HarithaVattikuti merged commit c990e6d into actions:main May 22, 2024
20 checks passed
@RobPasMue
Copy link

I believe there is some issue with the artifacts uploaded in the last release... see actions/setup-python#886 (comment) and the related issue actions/setup-python#886

IMO, the update of the upload-artifact action might have caused this (?)

mayeut added a commit to mayeut/python-versions that referenced this pull request Jun 10, 2024
actions#267 introduced a regression in asset upload.

The layout of the artefacts to upload has changed. They're now all flatten in the working directory whereas before that PR, only the `hashes.sha256` file was directly in the working directory.
The change in layout caused `.toString()` to be called on python archives which corrupts the data that gets uploaded (as can be seen by comparing sha256 in `hashes.sha256` with the ones from other uploaded assets).

This commit ensures that a "flat layout" is used and that `toString()` is only called for `hashes.sha256`.
@mayeut mayeut deleted the update-actions branch June 20, 2024 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants