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

Allow adding the URL of the PR to the release notes #582

Merged
merged 4 commits into from Jul 2, 2020

Conversation

b00men
Copy link
Contributor

@b00men b00men commented Jul 1, 2020

We are actively uses a repo's release notes in scope of other repos.
This makes a github PR's short link (#NUMBER) lead to a wrong place.

So I would like to have an option to use an issue's complete URL instead of hardcoded replacements.

Copy link
Member

@jetersen jetersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, one question 👏

lib/commits.js Outdated
@@ -35,6 +35,7 @@ module.exports.findCommitsWithAssociatedPullRequestsQuery = /* GraphQL */ `
nodes {
title
number
url
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we do the same include trick as we did for body? Again to avoid pulling extra data that is not needed.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It makes sense

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@jetersen jetersen merged commit 842a9a8 into release-drafter:master Jul 2, 2020
@jetersen
Copy link
Member

jetersen commented Jul 2, 2020

@b00men marvelous, thanks for your contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants