Skip to content

fix: Don't fail on Alpine checksums for Security releases#2421

Open
nschonni wants to merge 1 commit intonodejs:mainfrom
nschonni:skip-alpine-security-failure
Open

fix: Don't fail on Alpine checksums for Security releases#2421
nschonni wants to merge 1 commit intonodejs:mainfrom
nschonni:skip-alpine-security-failure

Conversation

@nschonni
Copy link
Member

Description

Added back the recent "skip" concept for the Apline checksums falling over, but still allow Alpine to try and update itself if the checksum is available.

Motivation and Context

Testing Details

Example Output(if appropriate)

Types of changes

  • Documentation
  • Version change (Update, remove or add more Node.js versions)
  • Variant change (Update, remove or add more variants, or versions of variants)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (none of the above)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING.md document.
  • All new and existing tests passed.

@MikeMcC399
Copy link
Contributor

MikeMcC399 commented Mar 16, 2026

@nschonni

We've overlapped again! So I'll put my help text PR #2420 into draft state.

@nschonni nschonni force-pushed the skip-alpine-security-failure branch from b302c67 to 678ef04 Compare March 16, 2026 14:45
@nschonni
Copy link
Member Author

I've updated the help text after seeing yours

@nschonni
Copy link
Member Author

I didn't check what happens to the automation after a partial version update without the Alpine builds lands here? Will it still pick up the partial/continue with a new PR (with a colliding PR name)

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.

2 participants