Skip to content

Exclude PRs merged but not in releasing branch #50

Closed

Description

@minrk

Is your feature request related to a problem? Please describe.

Not all merged PRs are merged into the currently to-be-released branch. It would be useful to include only those that are merged into the current branch.

Describe the solution you'd like

Specify a branch I am releasing from (default: current branch or repo default), and only include PRs that have been merged into that branch.

The simplest solution is to look at the base branch in the pr metadata. This, however, will not include multi-stage PRs (multiple PRs to a feature branch, which is later merged to main). It's a good start, though.

It is possible from the git graph, though, to determine exactly which PRs are ancestors to the current branch. That might be more work than we want to do, and can always be a second improvement later.

Additional context

I'm working on a jupyerhub release, and we currently have a long-running 'rbac' branch with merged PRs that are not being released. I have to manually remove those from the generated changelog.

In the future, though, I'd like all of those to be included as changelog items, rather than the one big rbac->main merge at the end.

Activity

  1. welcome commented on Apr 9, 2021

    @welcome

    Thanks for opening your first issue here! Engagement like this is essential for open source projects! 🤗

    If you haven't done so already, check out EBP's Code of Conduct. Also, please try to follow the issue template as it helps other community members to contribute more effectively.

    If your issue is a feature request, others may react to it, to raise its prominence (see Feature Voting).

    Welcome to the EBP community! 🎉

  2. changed the title [-]Exclude PRs merged into feature branches[/-] [+]Exclude PRs merged but not in releasing branch[/+] on Apr 9, 2021
  3. choldgraf commented on Apr 9, 2021

    @choldgraf
    Member

    I think this is a good idea - agreed that we can go with an 80/20 principle here and focus on the low hanging fruit first!

  4. blink1073 commented on Apr 14, 2021

    @blink1073

    cf #42

  5. minrk commented on Feb 27, 2023

    @minrk
    CollaboratorAuthor

    closed by #42

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions