-
Notifications
You must be signed in to change notification settings - Fork 175
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
docs: remove references to make bump in releasing #12222
Conversation
Update for how we use tags instead of make bump.
```shell | ||
make bump project=${project} version=${version} | ||
${projectPrefix}${projectVersion} |
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.
${projectPrefix}${projectVersion} | |
git tag -a ${projectPrefix}${projectVersion} -m 'chore(release): ${projectPrefix}${version}' |
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.
No, this defines the name of the tag, not the command itself. Then there's examples of tags. The commands to use are inline in the instructions.
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.
ok, then it may be worth calling out that the tag message should include the project prefix too right (e.g. chore(release): [email protected]
)?
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.
we haven't previously been doing that (and also ot
isn't a project prefix - flex uses ot3
, robot stack uses v
)
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.
nioce
Codecov Report
@@ Coverage Diff @@
## edge #12222 +/- ##
==========================================
- Coverage 74.01% 73.70% -0.31%
==========================================
Files 2193 1476 -717
Lines 60358 48416 -11942
Branches 6223 2978 -3245
==========================================
- Hits 44674 35687 -8987
+ Misses 14254 12276 -1978
+ Partials 1430 453 -977
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
🧜
* edge: refactor(protocol-engine): Shorten error docstrings (#12230) chore(app): update electron-builder version from v24 alpha to v24 (#12233) chore(app): add new icons for ODD hi-fi (#12231) docs: more release docs (#12226) fix(app, odd): check isOnDevice true instead of if it is null (#12228) docs: remove references to make bump in releasing (#12222) refactor(app): scroll to current split from jump to tick, qol tweaks to run preview (#12218) fix(app): rewire cli and jupyter snippets to offsets modals (#12180) feat(app): handle non-deterministic protocols (#12173)
We changed to a direct tag-based version process and don't use
make bump
anymore (we don't even havelerna
installed) so let's remove the actualmake bump
target that doesn't work anymore and update RELEASING.md to contain the new information.