Skip to content

improves error handling #48

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

Merged
merged 7 commits into from
Feb 27, 2018
Merged

improves error handling #48

merged 7 commits into from
Feb 27, 2018

Conversation

dmalan
Copy link
Member

@dmalan dmalan commented Jan 7, 2018

  • Color-codes log
    • Green if successful
    • Yellow if valid syntax but constraint violation
    • Red if invalid syntax
  • Requires that *.db exist for SQLite, so that students don't accidentally create an empty DB
  • Catches SQLAlchemy exceptions, outputs only their error messages.
  • Implements check whether sqlite:/// file exists, throw exception if not #40.
  • Capitalizes comments for style

@dmalan
Copy link
Member Author

dmalan commented Jan 7, 2018

@crossroads1112 any concerns?

@dmalan dmalan merged commit 6b48d14 into develop Feb 27, 2018
@dmalan dmalan deleted the error-handling branch February 27, 2018 02:37
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.

2 participants