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

Include proper error handling and warnings #6

Closed
IQAndreas opened this issue Sep 13, 2013 · 2 comments
Closed

Include proper error handling and warnings #6

IQAndreas opened this issue Sep 13, 2013 · 2 comments

Comments

@IQAndreas
Copy link
Owner

Allow for better error handling for the following cases:

  • There is a problem with the config options
  • The GitHub API returns an error
  • The target issue already exists in the repository (should at least warn)

I also want to prompt the user "You are about to import all 42 issues into hello-world. Are you sure you want to continue?"

@IQAndreas
Copy link
Owner Author

@dliuzzi has already done this for warnings thrown by the GitHub API: https://github.com/dliuzzi/tools

All that needs to be done is filter through the commits and make the changes (I'm pretty sure this code is too far-gone to do a straight up merge).

@IQAndreas
Copy link
Owner Author

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

No branches or pull requests

1 participant