search

getsentry / sentry-dotnet

Sentry SDK for .NET
https://docs.sentry.io/platforms/dotnet
MIT License
604 stars 206 forks source link

ci: change updater PR strategy to update #3780

Closed vaind closed 3 days ago

vaind commented 5 days ago

The default (which was the previous behaviour when the udpater was created) creates new branch & PR for every release.

The "update" strategy updates a single PR per dependency.

jamescrosswell commented 2 days ago

Not sure I get this. Why does this matter? So what problem does changing to update strategy solve?

vaind commented 2 days ago

Not sure I get this. Why does this matter? So what problem does changing to update strategy solve?

less maintenance when you need don't want to merge a dependency yet but need to wait for another release. Also applies when you just don't care about the update because it doesn't change anything for the consuming SDK. You can leave the PR open and it will get updated the next time there's a release on the dependency, instead of creating multiple PRs, one for each version.

jamescrosswell commented 2 days ago

You can leave the PR open and it will get updated the next time there's a release on the dependency, instead of creating multiple PRs, one for each version.

Ahhh, I see. Cool, thanks @vaind !