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

Additional info on checker errors #49

Merged
merged 2 commits into from
Dec 16, 2019
Merged

Additional info on checker errors #49

merged 2 commits into from
Dec 16, 2019

Conversation

larsk21
Copy link
Contributor

@larsk21 larsk21 commented Dec 9, 2019

  • fix: add missing checkTimeOverlap call

  • format specifiers in CheckerErrorMessage

  • additional info when creating checker errors

  • remove return on error (multiple errors possible)

  • tests for multiple errors

resolves #45

- fix: add missing checkTimeOverlap call

- format specifiers in CheckerErrorMessage
- additional info when creating checker errors
- remove return on error (multiple errors possible)
- tests for multiple errors

resolves #45
@larsk21 larsk21 requested a review from gstuer December 9, 2019 15:05
gstuer
gstuer previously approved these changes Dec 16, 2019
@gstuer gstuer merged commit 073a01d into master Dec 16, 2019
@gstuer gstuer deleted the PR_AdditionalErrorInfo branch December 16, 2019 13:08
gstuer added a commit that referenced this pull request Apr 19, 2020
Additions/Improvements:
- Added version command #50 
- Added more informative error messages #49 #81 
- Added schemata for the different JSON files #56 
- Added day time exceedance check #60
- Improved vacation mechanics #79 
- Improved stability of LaTeX generator #64 #72
- Improved stability of JSON parser #68

Bugfixes:
- Fixed LaTeX special characters not being escaped #69
@gstuer gstuer mentioned this pull request Apr 19, 2020
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.

CheckerError providing additional information to user
2 participants