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
feat: --minify
option (#1120)
#1121
Conversation
@domoritz, this is it! |
Please fix the tests |
Are that tests broken because of my commit? |
Some of them. I forgot that i need the binary built before running the tests. Should i just add an |
We already do in
|
Oh, i meant the |
Head branch was pushed to by a user without write access
Why do we need to link? Can't we call the correct file my path? |
Well, maybe. On my computer it needs chmod before it works, so I thought a link would be better. |
Ha, it didn't work because it is calling jest directly, not the npm |
use I don't like to link without the user knowing that we are doing it. It could cause problems if people have multiple clones. |
It didn't work on windows... Do you know why? |
No. I don't use windows. |
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 good. Please update the docs at https://github.com/vega/ts-json-schema-generator#options.
Anything more? |
Thank you |
Fixes #1120
Version
Published prerelease version:
v0.98.0-next.0
Changelog
🎉 This release contains work from new contributors! 🎉
Thanks for all your work!
❤️ Arthur Fiorette (@arthurfiorette)
❤️ Mark Sheinkman (@MarkSheinkman)
❤️ null@chimurai
❤️ Kari Lavikka (@tuner)
🚀 Enhancement
--minify
option (Add an option to minify the output. #1120) #1121 (@arthurfiorette)🐛 Bug Fix
removeUnreachable
#1048 (@tuner)🔩 Dependency Updates
Authors: 7