Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: when a pull request is already open, link to it #109

Merged
merged 2 commits into from
Jan 27, 2019

Conversation

jakebolam
Copy link
Contributor

No description provided.

@jakebolam jakebolam merged commit 90841af into master Jan 27, 2019
@jakebolam jakebolam deleted the fix/fix-pull-reuqest-already-open branch January 27, 2019 00:24
@all-contributors-release-bot
Copy link
Member

🎉 This PR is included in version 1.0.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Comment on lines +169 to +174
// Hard fail, but recoverable (not ideal for UX)
this.log.error(error)
throw new AllContributorBotError(
`A pull request is already open for the branch \`${branchName}\`.`,
)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

@jakebolam I know it's been a while, but do you by chance recall what this code was about? It's not tested, and the error message doesn't make sense to me

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants