Closed hpeebles closed 2 months ago
Hey @hpeebles
sounds good, thanks for the PR. I think you missed updating the yarn lock (currently not on my machine so I can't do it). Feel free to add another commit and we get this merged quickly. Also, please do the required steps for the automated github ci npm release (bump version, write a changelog entry) for inspiration check the changes made in this PR
I did not create a PR template file here yet, sorry. The template can be found in schema2typebox here
thanks!
could you make the release 1.6.1 instead (also adapt changelog please) and remove the ^ from the devDependencies
will do this on monday if you have no time, thanks!
I think that's everything sorted!
I'm currently getting issues when pulling ts2typebox in as a dependency to another project because it has a fixed dependency on Typescript 5.2.2.
This PR makes it avoid using fixed dependency versions so that there will be fewer compatibility issues.