Tweak release drafter template and categories#175
Conversation
Turnerj
left a comment
There was a problem hiding this comment.
Looks good to me! Didn't think about the situation of dependency updates in our test projects.
|
Looks like the new dependabot integration in GitHub might allow us to have specific labels set for dependencies depending on the folder path. See here about being able to configure labels: https://help.github.com/en/github/administering-a-repository/customizing-dependency-updates#about-customizing-dependency-updates Their example doesn't specifically say you can have the one package manager listed twice however with how they configured the YML, it looks like it might support it. This would mean then we could have something like this: version: 2
updates:
- package-ecosystem: "nuget"
directory: "/Source/"
labels:
- dependencies
- minor
schedule:
interval: "daily"
- package-ecosystem: "nuget"
directory: "/Tests/"
labels:
- dependencies
schedule:
interval: "weekly"This is still just a theory at the moment however it looks like it might work. |
|
Oh cool, I didn't know this was possible. A few questions:
This is not going to be foolproof but I guess it might give a better defaults and save some time but also wondering if it's worth it for the |
|
Yeah, while we would have some extra control like being able to divide "test" package labels vs "source" package labels, we don't get the fine details of whether the label is for patch updates, minor version updates or major version updates. For my example, Really our versions numbers ideally would go up on our changes alone. If we add new features in a backwards compatible way, that's |
|
In that case, I'm not sure adding this would be too valuable. We're always going to have to give dependabot a bit of thought and mess with it's labels. Nice to know this exists though. |
@dependabot-bot marks it's PR's with the
dependencieslabel. I marked these as 'New Features' in release drafter. However, unit test NuGet package updates should show up in 'Maintenance' instead. This PR will make it so that each and ever PR will require manual labelling, so they appear in the correct section.