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

Validate config.ini in Config.ts and add tests to ensure schema can't be deviated from #25

Open
IncPlusPlus opened this issue Dec 18, 2020 · 0 comments
Open

Validate config.ini in Config.ts and add tests to ensure schema can't be deviated from #25

IncPlusPlus opened this issue Dec 18, 2020 · 0 comments
Assignees
@IncPlusPlus
Labels
enhancement New feature or request
Milestone
1.2.0

Comments

@IncPlusPlus
Copy link
Owner

It's be good to have a schema to validate against. Currently, a blank ini file will be read just fine but will cause runtime errors when any value is accessed as it won't exist.
@IncPlusPlus IncPlusPlus added this to the 1.1.1 milestone Dec 18, 2020
@IncPlusPlus IncPlusPlus self-assigned this Dec 18, 2020
@IncPlusPlus IncPlusPlus changed the title Validate config.ini in Config.ts and add tests Validate config.ini in Config.ts and add tests to ensure schema can't be deviated from Dec 20, 2020
@IncPlusPlus IncPlusPlus added the enhancement New feature or request label Jan 2, 2021
@IncPlusPlus IncPlusPlus modified the milestones: 1.1.1, Future, 1.2.0 Jan 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IncPlusPlus IncPlusPlus

Labels
enhancement New feature or request
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
1 participant
@IncPlusPlus