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

generation of schema from typescript #21

Draft
wants to merge 35 commits into
base: main
Choose a base branch
from

Conversation

Mearman
Copy link

@Mearman Mearman commented Mar 14, 2024

kguillouard and others added 18 commits March 12, 2024 00:38
…pdates. Adjusted file associations in VSCode settings.
- Added a new TypeScript file `generate.ts` to handle the generation of JSON schema from a given TypeScript file. The script validates input parameters, generates a UUID, creates the schema configuration, and writes the resulting JSON schema to an output file.
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
@chainlist
Copy link

I did some modifications (mostly adding some description + pattern on color string) so you might want to pull in case your MR is merged before mine (if it happens)

chainlist and others added 11 commits March 15, 2024 16:20
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
chainlist and others added 3 commits March 15, 2024 16:21
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
Co-authored-by: Olivier Mengué <[email protected]>
@Mearman
Copy link
Author

Mearman commented Mar 15, 2024

thanks @chainlist. that was why I kept it as a draft

@chainlist
Copy link

chainlist commented Mar 15, 2024

I really wonder if having the TS type file in this repository is a good thing.
JSONSchema doesn't bother me that much because it is part of the specifications for a JSON file.

Would, adding the TS type in https://github.com/DefinitelyTyped/DefinitelyTyped be a bit better ?
so doing the following:

npm install @types/jsoncanvas

would work.

Really asking what would be better.

@Mearman
Copy link
Author

Mearman commented Mar 15, 2024

@chainlist from playing quite large and unwieldy schemas, being able to manipulate it in TypeScript makes things considerably easier, and more accessible to contributors.

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.

2 participants