-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Option to use double quotes (") as escape character #476
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #476 +/- ##
==========================================
+ Coverage 98.48% 98.52% +0.03%
==========================================
Files 9 9
Lines 661 676 +15
Branches 266 264 -2
==========================================
+ Hits 651 666 +15
Misses 10 10 ☔ View full report in Codecov by Sentry. |
README.md
Outdated
* If set to true, will use double quotes for strings | ||
* @default false | ||
*/ | ||
readonly doublequoteEscape: boolean; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great can you remove the word readonly here? I'll get this released for you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great can you remove the word readonly here? I'll get this released for you
done! thanks!
change released https://github.com/mrjono1/joi-to-typescript/releases/tag/4.14.0 |
Thankyou for the contribution |
thanks @mrjono1 ! |
Our team is using double quotes for strings and we have to manually edit generated interfaces to pass our linter. This PR adds
Settings.doublequoteEscape
which will use double quotes in generated interfaces if set totrue
. Default behavior stays unchanged.