Skip to content
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

ci(release-please): change PR title and ensure git tag consistency #7153

Merged
merged 3 commits into from
Jun 9, 2023

Conversation

benelan
Copy link
Member

@benelan benelan commented Jun 7, 2023

Related Issue: #6988

Summary

This PR changes the release-please PR title to chore: release latest. The default is chore: release master.

It also makes sure a v isn't added before the version in git tags. Their schema doc says it doesn't by default,
but I'm not convinced (see links below) so I want to hardcode the config value. The v being in the tag wouldn't break anything in our CI, but Lerna doesn't add a v and we should make sure our tag format is consistent for our consumer's CI.

https://github.com/googleapis/release-please/blob/bdd9b0158ce79d958da01ebf54cb6b07925bc125/schemas/config.json#L77-L79

https://github.com/googleapis/release-please/blob/bdd9b0158ce79d958da01ebf54cb6b07925bc125/src/strategies/base.ts#L139

@benelan benelan requested a review from a team as a code owner June 7, 2023 23:17
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label Jun 7, 2023
Copy link
Member

@driskull driskull left a comment

Choose a reason for hiding this comment

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

👍

@benelan benelan added the skip visual snapshots Pull requests that do not need visual regression testing. label Jun 9, 2023
@benelan benelan merged commit 24bef80 into master Jun 9, 2023
@benelan benelan deleted the benelan/more-release-ci-tweaks branch June 9, 2023 23:30
@github-actions github-actions bot added this to the 2023 June patch priorities milestone Jun 9, 2023
benelan added a commit that referenced this pull request Jun 15, 2023
…import

* origin/master: (30 commits)
  chore: release next
  fix(radio-button, radio-button-group): prevent emitting events when selecting a checked radio button (#7102)
  feat(list): Add slots for filter actions (#7148)
  chore: release next
  fix(tooltip): improve component timing (#7172)
  chore: release next
  fix(input-date-picker): Fix showing the placeholder when resetting the value (#7156)
  chore: release next
  fix(input,input-number): allow typing decimal separator in firefox for arabic locale (#7173)
  build(deps): Bump workbox-build from 6.5.4 to 7.0.0 (#7167)
  build(deps): Bump type-fest from 3.5.0 to 3.11.1 (#7169)
  build(deps): Bump chromatic from 6.17.4 to 6.19.5 (#7171)
  build(deps): Bump @floating-ui/dom from 1.2.9 to 1.3.0 in /packages/calcite-components (#7168)
  build(deps): Bump @esri/calcite-ui-icons from 3.22.9 to 3.23.1 in /packages/calcite-components (#7166)
  build: update browserslist db (#7162)
  ci(release-please): change PR title and ensure git tag consistency (#7153)
  docs(changelog): Update commit message for #7160 (#7161)
  chore: release next
  fix(list): Improve border styling (#7160)
  chore: release next
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants