Update artifact and Pages actions to supported Node 24 releases - #30
Closed
acecchini wants to merge 3 commits into
Closed
Update artifact and Pages actions to supported Node 24 releases#30acecchini wants to merge 3 commits into
acecchini wants to merge 3 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The successful release dry run exposed Node 20 deprecation warnings in artifact actions. This updates upload/download-artifact and Pages actions to reviewed immutable Node 24 releases. Existing artifact names, extraction and release dependencies are preserved; the current downloader fails digest mismatches by default. Required docs validation now packages a distinct Pages artifact without deploying it.
Validation: hooks/actionlint and all implementation PR checks pass. The actual validation-only run https://github.com/acecchini/bearshape/actions/runs/34223582731 completed 38 jobs successfully and skipped publication. Installed consumers passed 1,101 tests per endpoint; the 46-file Pages archive was inspected. Node 20 warnings are gone; the latest upstream downloader's Buffer() deprecation notice is documented without suppression.
The final review checkout reproduces both tested artifacts exactly: wheel SHA256 c1806203da013c9eaf2482a309c686a984c0a7031a495efbd100a824536b57d2 and sdist SHA256 38d89d4ce1b351601ff8602a9a27b4e9af46db056be3ac0eaf41a68ac56a3706. That identical wheel also passed 95 real CuPy tests on both GPU endpoints. The handoff report records the final evidence and remaining contract/admin decisions.
Stacked on #29. No actual Pages/PyPI deployment, merge, ownership transfer or settings change occurred.
Integration record: this reviewed head is preserved in main through PR #31, merge commit df81f00. The owner approved merging this work. This dependent PR is closed as integrated; its commits were not squashed or discarded.