Skip to content

Embed JSON schemas in code #105

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 3 commits into from
Dec 14, 2020
Merged

Embed JSON schemas in code #105

merged 3 commits into from
Dec 14, 2020

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Dec 13, 2020

Previously, the JSON schemas were provided as external files. This presented some difficulties for testing and was also
likely to make distribution and installation more complex.

By embedding the schemas as binary data in the Go code, the
tool can consist of a single standalone binary.


Fixes #95 (Thanks Silvano!)

Previously, the schemas were provided as external files. This presented some difficulties for testing and was also
likely to make distribution and installation more complex. By embedding the schemas as binary data in the Go code, the
tool can consist of a single standalone binary.
The previous `go fmt` command was not formatting the code under the testdata folder.
The stringer tool is used to generate stringers for the "enums".
@per1234 per1234 force-pushed the per1234/embed-schemas branch from 9f86398 to 3aa02c7 Compare December 14, 2020 14:42
@per1234 per1234 merged commit ece3ff1 into main Dec 14, 2020
@per1234 per1234 deleted the per1234/embed-schemas branch December 14, 2020 14:52
@per1234 per1234 mentioned this pull request Dec 17, 2020
@per1234 per1234 added topic: code Related to content of the project itself type: enhancement Proposed improvement labels Sep 29, 2021
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to write tests for schema-based check functions
2 participants