-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Upgrade json-schema-to-typescript
#7934
Comments
Thank you! Closed by #7938 |
wow! amazing! thank you for the super fast turnaround on this the other one I was going to ask about was |
No, thank you for bringing this to our attention! Also just dug around our package.json a bit more - managed to also decrease the total install size of the
This one will be trickier. Definitely a breaking change since we expose our logger, including the pino options, to our users. This will be especially bad for users that add their own pino transports. For example, I'm using a pino transport to automatically upload my logs to axiom. An alternative logging library would also need to produce JSON logs (which is essential for being able to filter logs). + pino claims to have a very low performance overhead - would have to make sure an alternative library doesn't decrease speed in favor of dependency count. |
An easier one to tackle might be bcherny/json-schema-to-typescript#623. The author of that library was initially reluctant, but said if there was significant interest from others, they'd be open to making the change I was also curious why pinned package versions are used as prefixing all packages with
Amazing!! Definitely tell your boss 😆 Seriously I looked at all the CMS and payload was already the lightest and I think it can get even further ahead, which will be a great line in the release announcement
Thanks for sharing! That makes sense. Maybe the easier thing then would be to see if |
Crazy! Hoping for that to be resolved soon. Lmk if I can help make a PR for this library, but I'd like to hear what the author has to say first regarding which glob library they prefer. One thing we'll be looking into is replacing the
We've experienced a lot of stability issues in our package.json due to carets. We've had packages we depend on release broken updates or make breaking changes (sometimes masked as bug fixes) in patch versions. To mitigate this, we've started specifying exact package versions to maintain better control over our dependencies. Well, we still face potential issues with peer dependencies of our dependencies, which we can't control, but this issue has been happening a lot less since.
Thank you! Yea a lot of effort has been put into keeping the core A good example that may be interesting for you given that you're a Svelte person: While payload's admin panel (
We should definitely issue in their repo about this. The maintainer seems to be very conscious about performance and all that. |
That's great news! Yeah, that and
Yeah, that sounds really interesting to explore! If you or anyone else were interested in talking about it at the Svelte Summit virtual conference in October, they're currently accepting ideas for talks for the next two weeks: https://www.sveltesummit.com/2024/fall (I don't see the deadline on the website, but believe it's Sept 14) |
Both deep-equal and qs are already replaced! (Be sure to check the
Sounds interesting, I'll bring this up with our in-house influencer @jmikrut ! |
This issue has been automatically locked. |
Link to reproduction
https://npmgraph.js.org/[email protected]
Environment Info
Describe the Bug
Upgrading would drop 14 dependencies:
https://npmgraph.js.org/[email protected] - 51 dependencies
https://npmgraph.js.org/?q=json-schema-to-typescript - 37 dependencies
Reproduction Steps
https://npmgraph.js.org/[email protected]
Adapters and Plugins
No response
The text was updated successfully, but these errors were encountered: