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.

Sign up for GitHub

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

Jump to bottom

[Suggest] Add protection against wrongly formatted json files to ensure usage of dever is still possible #48

Closed
czprz opened this issue Mar 28, 2022 · 0 comments · Fixed by #54
Closed

[Suggest] Add protection against wrongly formatted json files to ensure usage of dever is still possible #48

czprz opened this issue Mar 28, 2022 · 0 comments · Fixed by #54
Assignees
@czprz
Labels
Suggestion Idea/Feature suggestion
Milestone
0.10.0

Comments

@czprz
Copy link
Owner

czprz commented Mar 28, 2022

Is your feature request related to a problem? Please describe.
When one or more dever.json files are formatted incorrectly. Dever becomes useless due to errors

Describe the solution you'd like
A protection against 'corrupt' dever.json files. By checking them and then not including them if they're invalid.
@czprz czprz added the Suggestion Idea/Feature suggestion label Mar 28, 2022
@czprz czprz added this to the 1.0.0 milestone Mar 29, 2022
@czprz czprz self-assigned this Apr 12, 2022
czprz added a commit that referenced this issue Apr 12, 2022
@czprz
#48 Replaced rethrowing of exception with console.error indicating fi…
9ccc472 
…lepath with problems instead of making dever unusable
@czprz czprz mentioned this issue Apr 12, 2022
Merged
@czprz czprz closed this as completed in #54 Apr 12, 2022
czprz added a commit that referenced this issue Apr 12, 2022
@czprz
Merge pull request #54 from czprz/48-suggest-add-protection-against-w…
66a92a3 
…rongly-formatted-json-files-to-ensure-usage-of-dever-is-still-possible

#48 Fix issue with dever.json incorrectly formatted causing dever not being usable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@czprz czprz

Labels
Suggestion Idea/Feature suggestion
Projects
None yet
Milestone
0.10.0
Development

Successfully merging a pull request may close this issue.

#48 Fix issue with dever.json incorrectly formatted causing dever not being usable czprz/dever
1 participant
@czprz