Closed yurishkuro closed 10 months ago
I'm a beginner to github actions but I want to work on this issue. I am learning about github actions online. May you please guide how to learn and solve this issue?
Hi @yurishkuro, How are you doing? I saw no one is working on this issue so can you please assign it to me so that i can work on this issue?
Yeah thanks, Now I'm doing this!
Hey @Suraj-kumar00 are you currently working on this ?
@yurishkuro Can you please explain " to validate that all PRs have changelog labels" means ? https://github.com/jaegertracing/jaeger/pull/4735/commits/a44a2452219f9ed246892a633bee3f0c6fa686dc
As this was the only PR with changelog. So I am definitely missing something here. Please explain.
@devidasjadhav We are trying to ensure that each PR has a changelog label, which would allow us to generate release notes more easily and in an automated way. Currently, the release process generates the list, but requires some manual processing of the ones which are not tagged.
A couple pointer is looking at this existing action: https://github.com/marketplace/actions/pr-labels and this repo : https://github.com/actions/labeler which is using these actions too.
The current release notes script is located here too: https://github.com/jaegertracing/jaeger/blob/main/scripts/release-notes.py
Hey @Suraj-kumar00 are you currently working on this ?
Yes.
I just ran the script and it shows all dependabot PRs under Uncategorized, instead of skipping them.
BTW, instead of building special logic for dependabot, we can change its config to add changelog:skip
label (now it only sets dependencies
label).
cc @anshgoyalevil
@yurishkuro Did you used the --skip-dependabot
flag?
We can change the dependabot labelling settings but that would defeat the purpose of --skip-dependabot flag
Ha, I didn't know we had that flag. Well if anything it should be --include-dependabot
, so that it's off by default, but we never really include those in the changelog anyway.
Yes, then a better workaround here would be to rename the --skip-dependabot
to --include-dependabot
arg, and use it accordingly, but since https://github.com/jaegertracing/jaeger/pull/4854 solves this purpose anyhow, do we need to proceed with this arg renaming thing?
No, I am good with the current state
We need to use these new scripts in jaeger-ui repo as well.
We already tried this in the past, but there was no enforcement and it didn't take.
changelog:***
labels