Skip to content

Conversation

@cameel
Copy link
Collaborator

@cameel cameel commented Oct 10, 2025

Just a bunch of refactors extracted from the upcoming PR that actually updates the logic.

The changes either have zero effect on behavior or only affect it in trivial ways, like giving the temporary dir a recognizable prefix or enabling Bash error handling.

@cameel cameel force-pushed the prerelease-logic-cleanup branch from 8dc6dea to 9bc0122 Compare October 10, 2025 19:41
Copy link
Collaborator

@nikola-matic nikola-matic 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, although another set of eyes would be nice. Have you run this locally?

Copy link
Member

@clonker clonker left a comment

Choose a reason for hiding this comment

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

I have checked scripts/create_source_tarball.sh as well as scripts/build.sh locally and it works. For the others we would have seen failure in CI. Looks good to me. I found the prerelease.txt handling confusing, especially in light of the "new" release vs prerelease vs nightly situation. But the PR is good to go in for all I am concerned with #16246 as a follow-up.

@nikola-matic nikola-matic merged commit 94254b0 into develop Oct 14, 2025
75 checks passed
@nikola-matic nikola-matic deleted the prerelease-logic-cleanup branch October 14, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants