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

Add simple dever json validation #55

Merged
merged 25 commits into from
Apr 12, 2022
Merged

Conversation

czprz
Copy link
Owner

@czprz czprz commented Apr 12, 2022

Describe your changes

Added simple dever.json validation which initially will only check for correctly formatted JSON and that pre-defined keys are not being used as keywords

Issue ticket number and link

Initial implementation of new functionality for validating dever.json at location or by passing file path location

Checklist before requesting a review

✔️ I have read and followed guidelines for contributing to this repository
✔️ I have performed a self-review of my code
✔️ I have made sure to create a pull request from a topic/feature/bugfix branch
✔️ I have followed the commit's or even all commits' message styles matches our requested structure.

czprz and others added 21 commits January 31, 2022 23:25
# Conflicts:
#	bin/configuration/handleComponents.js
+ Removed console message when project cannot be found. As this should not be part of the config handler
+ Added new simple implementation of dever.json validate command
+ Created constant helper to have a central location for predefined keys
#27 Ensure project keywords does not conflict with pre-defined keys
#4 Moved location config file to user folder to avoid it being cleared by npm update
# Conflicts:
#	package.json
@czprz czprz added the enhancement New feature or request label Apr 12, 2022
@czprz czprz added this to the 0.10.0 milestone Apr 12, 2022
@czprz czprz self-assigned this Apr 12, 2022
@czprz czprz merged commit fc18a57 into main Apr 12, 2022
@czprz czprz deleted the Add_simple_dever_json_validate branch April 12, 2022 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant