-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Flow 8.1.12 was accidentally tagged from the 9.0 branch #3355
Comments
Both I'd say |
Doh.
Could it be those fail, because the release helpers are not in the 8.1 branch? Or something like that? |
No, actually only 8.1.12 is on the 9.0 branch, phew! The |
Something IS very weird here though: Note this is about the flow-development-distribution as we checkout that one which has the build scripts:
But looking at this mentioned commit on GH suggests it only exists in the 8.3 branch 🤔 how can that be, unless 8.1 and 8.2 are broken? |
Oh wait, I guess the problem is "simple" in that we composer install failed and we didn't checkout the build essentials package?
|
Those scripts should definitely exit with an error code when something goes wrong, and Jenkins should then fail the build! |
aaand so this will run https://github.com/neos/flow-development-distribution/blob/8.1/Build/release.sh but fail for the whole tagging process beecause the essentials are not there, and tehn in the end call GH API to create a release which I guess GH then just creates on the default branch if the tag doesn't exist. |
yep, I guess that solves the riddle. needs more failure checks, all else seems good. |
Flow 8.1.12 was tagged on the 9.0 branch. 🙈
I edited the text at https://github.com/neos/flow-development-collection/releases/tag/8.1.12
Being the latest 8.1, what should we do? Drop it, or release 8.1.13 from the right branch?
The paramaters of the release job were correct (see https://jenkins.neos.io/job/flow-release/444/parameters/), but the job failed silently (see https://jenkins.neos.io/job/flow-release/444/console).
The text was updated successfully, but these errors were encountered: