You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The code above means "Static release branch set in TagBot config if it's set, otherwise the repo's default branch". It's ignoring release branch info from a Registrator invocation like @JuliaRegistrator register branch=foo.
The actual effect of this is we miss out on the nice "N commits since this release" for those latter releases.
The priority for release branch selection should be:
Branch specified by Registrator invocation
Release branch specified in TagBot config
Default branch
Right now we're just missing (1). We can look up the commit comment by reading the registry PR body, but we're out of luck for the web UI.
I won't get to this soon but I'll keep it in mind for the rewrite ™️
The text was updated successfully, but these errors were encountered:
TagBot/tagbot/action/repo.py
Lines 151 to 154 in bb54f2e
The code above means "Static release branch set in TagBot config if it's set, otherwise the repo's default branch". It's ignoring release branch info from a Registrator invocation like
@JuliaRegistrator register branch=foo
.The actual effect of this is we miss out on the nice "N commits since this release" for those latter releases.
The priority for release branch selection should be:
Right now we're just missing (1). We can look up the commit comment by reading the registry PR body, but we're out of luck for the web UI.
I won't get to this soon but I'll keep it in mind for the rewrite ™️
The text was updated successfully, but these errors were encountered: