Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: --find-matching-prs for commits without PR-URL #130

Merged
merged 1 commit into from
Jun 16, 2022

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Jun 15, 2022

So .. I want to use changelog-maker for a project that doesn't use PR-URL metadata, and also, it really is kind of annoying to have to put that in when GitHub knows what commit belongs to what PR and we now have a useful way to query for it.

This introduces --find-matching-prs=true as an argument, then uses a GraphQL query to make a best guess as to what PR the commit belongs to and fills in the blanks for it. Because this is done before label collection it should also mean that we get to collect labels for commits that don't have PR-URL too, which is neat.

(If someone wants to go back and fix up #73 now that this introduces a GraphQL pattern so we can use it to find labels, then that might be neat too).

@rvagg rvagg merged commit 0aa7a2e into main Jun 16, 2022
@rvagg rvagg deleted the rvagg/find-matching-prs branch June 16, 2022 07:37
github-actions bot pushed a commit that referenced this pull request Jun 16, 2022
## [3.2.0](v3.1.0...v3.2.0) (2022-06-16)

### Features

* --find-matching-prs for commits without PR-URL ([#130](#130)) ([0aa7a2e](0aa7a2e))

### Trivial Changes

* update node versions in Actions matrix (& enable macos) ([#131](#131)) ([2a87620](2a87620))
@github-actions
Copy link

🎉 This PR is included in version 3.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

renawolford6 added a commit to renawolford6/changelog-maker-javascript that referenced this pull request Nov 9, 2022
## [3.2.0](nodejs/changelog-maker@v3.1.0...v3.2.0) (2022-06-16)

### Features

* --find-matching-prs for commits without PR-URL ([#130](nodejs/changelog-maker#130)) ([0aa7a2e](nodejs/changelog-maker@0aa7a2e))

### Trivial Changes

* update node versions in Actions matrix (& enable macos) ([#131](nodejs/changelog-maker#131)) ([2a87620](nodejs/changelog-maker@2a87620))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant