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

Check github URLs for existing pull request links #22

Merged
merged 2 commits into from Apr 26, 2018
Merged

Check github URLs for existing pull request links #22

merged 2 commits into from Apr 26, 2018

Conversation

DASPRiD
Copy link
Contributor

@DASPRiD DASPRiD commented Apr 25, 2018

This PR introduces actual checking of generated pull request links for their existence. If it encounters a redirect, it will probe the given Location as well.

This PR follows the original behaviour of first checking a possibly provided package name. If a PR with that package name does not exist, the script bails out immediately. Next it will probe a link generated from the composer package name. If that one fails as well, it will resort to trying out all remote URLs supplied by git. If everything fails, an exception will be thrown.

@weierophinney weierophinney merged commit 729f57c into phly:master Apr 26, 2018
weierophinney added a commit that referenced this pull request Apr 26, 2018
Check github URLs for existing pull request links
weierophinney added a commit that referenced this pull request Apr 26, 2018
weierophinney added a commit that referenced this pull request Apr 26, 2018
weierophinney added a commit that referenced this pull request Apr 26, 2018
Forward port #22

Conflicts:
	CHANGELOG.md
@weierophinney
Copy link
Contributor

Thanks, @DASPRiD!

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

Successfully merging this pull request may close these issues.

None yet

2 participants