Skip to content

Conversation

@daniel-beck
Copy link
Member

Copy link
Contributor

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Thanks!

@MarkEWaite MarkEWaite added the developer Pull requests that merits a release for plugin developers label Feb 15, 2023
@jglick
Copy link
Member

jglick commented Feb 15, 2023

Also in #1761, #1762, #1763.

@jglick
Copy link
Member

jglick commented Feb 16, 2023

BTW I doubt we really need to bother filing these explicitly. From the PoV of the bom, these are just routine plugin updates that can be processed whenever by Dependabot (except in the rare case that a security patch introduces an API or changes behavior likely to be covered by downstream tests).

@daniel-beck
Copy link
Member Author

daniel-beck commented Feb 16, 2023

@jglick We started doing this because you requested it in #1496 (review). Did we misunderstand what you were asking for?

@jglick
Copy link
Member

jglick commented Feb 16, 2023

Heh, completely forgot about that. In that case it was important to update a set of plugins together, though not because they were part of a security update per se.

Not a problem to do these updates, especially if you already have a process for it. Just FYI that in the typical case that they are unrelated updates released on the same day, there is no particular need to bump them manually, or to do so in an atomic PR.

lemeurherve pushed a commit to lemeurherve/bom that referenced this pull request Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer Pull requests that merits a release for plugin developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants