Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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 force-pushed the update-actions branch from b09f300 to 57bb646 Compare May 4, 2024 13:39
@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
@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
Development

Successfully merging this pull request may close these issues.

5 participants