-
-
Notifications
You must be signed in to change notification settings - Fork 384
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
Introduce new "v" version format #2374
Conversation
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2374 +/- ##
==========================================
- Coverage 62.82% 62.77% -0.06%
==========================================
Files 204 205 +1
Lines 19311 19283 -28
==========================================
- Hits 12132 12104 -28
Misses 6119 6119
Partials 1060 1060
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the workflow requires the regex starting with the v
prefix. This is good if we're 💯 sure that we don't need to release any patch version for the old releases tagged without such a prefix.
IMO I don't think we need to release any patch version for the old release, therefore we can make v
prefix mandatory also in the install.sh
we'll keep 0.35.x release branch name. We'll use "v" only for the tag, this way the documentation branches will work just like before, we are using "v" only for the github tag this way we are semver compliant
From Arduino CLI `>=0.35.0-rc.1`, the `v` prefix is expected in the GitHub release name. Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
From Arduino CLI `>=0.35.0-rc.1`, the `v` prefix is expected in the GitHub release name. The asset filenames do not contain the prefix. Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
From Arduino CLI `>=0.35.0-rc.1`, the `v` prefix is expected in the GitHub release name. The asset filenames do not contain the prefix. Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: dankeboy36 <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Ref: arduino/arduino-cli#2374 Signed-off-by: Akos Kitta <[email protected]>
Please check if the PR fulfills these requirements
See how to contribute
before creating one)
our contributing guidelines
UPGRADING.md
has been updated with a migration guide (for breaking changes)configuration.schema.json
updated if new parameters are added.What kind of change does this PR introduce?
Infrastructure change
What is the current behavior?
Fix #2341
What is the new behavior?
We release Arduino CLI with "v" version format
We still use
0.35.x
format for the doc branch and internally in the CLI the format is semver-compliant.Test run after pushing
0.35.x
branch: https://github.com/umbynos/arduino-cli/actions?query=branch%3A0.35.xTest run of
release-go-task
after pushingv.0.35.0
tag https://github.com/umbynos/arduino-cli/actions/runs/6576137988Does this PR introduce a breaking change, and is titled accordingly?
Technically no
Other information