Skip to content

fixed #2411 and typothing in script/quicktype comments: doesn't exis…#1

Open
feefladder wants to merge 23 commits intomasterfrom
true-schema
Open

fixed #2411 and typothing in script/quicktype comments: doesn't exis…#1
feefladder wants to merge 23 commits intomasterfrom
true-schema

Conversation

@feefladder
Copy link
Copy Markdown
Owner

Mainly to run tests

feefladder and others added 23 commits September 20, 2023 17:38
- fix type inference
- output more idiomatic @effect/schema code
…alues

fix: Expose the prefer-const-values Typescript language setting via CLI
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
* move omit empty check

* Remove unneeded import

---------

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
…ps#2476)

* make private members of TypeScriptZodRenderer protected

* add protected access for typeMapTypeForProperty, typeMapTypeFor

---------

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
typecheck PRs

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
…#2397)

Co-authored-by: Gudjon Ragnar Brynjarsson <gudjon.brynjarsson@controlant.com>
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
* !W(Core.Language.CSharp) add `keepPropertyName`

* !B(Core.Language.CSharp) `keywords`

* !T(Core.Language.CSharp) `keepPropertyName` default value is false and use ternary

---------

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
Enhance clarity and informativeness of error messages in the C++ code
generator for enum type conversions. Update the default case in
`to_json` function to include the specific enumeration name and the
unexpected value in the error message. This aids in quicker debugging
and aligns with best practices in error messaging.

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
…lideapps#2478)

Co-authored-by: David Siegel <dvdsgl@users.noreply.github.com>
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.