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

Rescue another NPE that may stop the whole loop #1

Merged
merged 1 commit into from Dec 14, 2018

Conversation

Projects
None yet
2 participants
@amatsuda
Copy link
Contributor

amatsuda commented Dec 14, 2018

extract_paths_from_https causes an unrescued NPE when the URI does not include enough number of /s .
This patch rescues the error and just skips processing that repo in such case.

@koic koic merged commit 3869a91 into koic:master Dec 14, 2018

@koic

This comment has been minimized.

Copy link
Owner

koic commented Dec 14, 2018

I was not aware of it and this patch is reasonable. Thank you!

@koic

This comment has been minimized.

Copy link
Owner

koic commented Dec 17, 2018

I released ghq_transfer 0.3.0 including this patch 🚀

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