Skip to content

feat: add skip-changelog flag#2535

Merged
chingor13 merged 4 commits intogoogleapis:mainfrom
stan-stately:stan/skip-changelog
Jun 5, 2025
Merged

feat: add skip-changelog flag#2535
chingor13 merged 4 commits intogoogleapis:mainfrom
stan-stately:stan/skip-changelog

Conversation

@stan-stately
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

Fixes #2007 🦕

Hi. I'm picking this up as I need it and no one ended up implementing it after it was discussed in #2007.

This is pretty simple. i just added the flag as proposed in the issue and propagated it into the strategies. this is not implemented for plugins - they have a different API. I think each must be done on a case-by-case basis

@stan-stately stan-stately requested review from a team as code owners May 12, 2025 07:11
@product-auto-label product-auto-label bot added the size: l Pull request size is large. label May 12, 2025
@asafhm
Copy link

asafhm commented May 25, 2025

Looking forward to this!

@chingor13 chingor13 changed the title feat: Add skip changelog flag feat: add skip-changelog flag Jun 5, 2025
@chingor13 chingor13 added this pull request to the merge queue Jun 5, 2025
Merged via the queue into googleapis:main with commit 2492658 Jun 5, 2025
9 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an option to disable changelog generation

3 participants