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

chore: release-please config #730

Merged
merged 1 commit into from
Jul 1, 2021
Merged

chore: release-please config #730

merged 1 commit into from
Jul 1, 2021

Conversation

joeldodge79
Copy link
Contributor

No description provided.

temp override changelog types for sdk-codegen-scripts for #727
set next versions on non-standard packages
@joeldodge79 joeldodge79 requested a review from jkaster July 1, 2021 20:47
@google-cla google-cla bot added the cla: yes label Jul 1, 2021
"packages/sdk-codegen": { "release-as": "21.0.18" },
"packages/sdk-codegen-scripts": { "release-as": "21.0.18" },
"packages/sdk-codegen": { "release-as": "21.0.19" },
"packages/sdk-codegen-scripts": { "release-as": "21.0.20", "changelog-sections": [{"type": "chore", "section": "Bug Fixes"}] },
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jkaster I just saw your reply - would you like me to undo this (no changelog entry) or leave it (changelog entry)?

Suggested change
"packages/sdk-codegen-scripts": { "release-as": "21.0.20", "changelog-sections": [{"type": "chore", "section": "Bug Fixes"}] },
"packages/sdk-codegen-scripts": { "release-as": "21.0.20" },

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should keep things as you had them and not accommodate my mistake

@joeldodge79 joeldodge79 merged commit 21e6c0a into main Jul 1, 2021
@joeldodge79 joeldodge79 deleted the rp-config branch July 1, 2021 21:14
Copy link
Contributor

@jkaster jkaster left a comment

Choose a reason for hiding this comment

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

I think it's fine for now but we should retain the behavior you had before and I'll use more accurate PR tags in the future.

"packages/sdk-codegen": { "release-as": "21.0.18" },
"packages/sdk-codegen-scripts": { "release-as": "21.0.18" },
"packages/sdk-codegen": { "release-as": "21.0.19" },
"packages/sdk-codegen-scripts": { "release-as": "21.0.20", "changelog-sections": [{"type": "chore", "section": "Bug Fixes"}] },
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should keep things as you had them and not accommodate my mistake

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants