Skip to content
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

Migrate type tests to TSTyche #507

Merged
merged 23 commits into from
Feb 23, 2024
Merged

Migrate type tests to TSTyche #507

merged 23 commits into from
Feb 23, 2024

Conversation

mrazauskas
Copy link
Contributor

@mrazauskas mrazauskas commented Feb 23, 2024

Closes: #506

Just to give you an idea how TSTyche type tests look in real life. Almost the same from outside, but the is a lot of difference internally.

For example, TSTyche uses typescript which is currently install in the repo. It can run test on any TypeScript in 4.x and 5.x series. Try out tstyche --target 5.0,current. Or tstyche --target next could be setup as a daily cron job in CI to test using TypeScript's nightly releases.

@FloEdelmann FloEdelmann added the dependencies Updates or changes to dependencies label Feb 23, 2024
@mrazauskas
Copy link
Contributor Author

@FloEdelmann This must be it. Ready for review!

Copy link
Owner

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, thanks, that looks perfect! I have just one small question regarding the new extra tsconfig.json file. Other than that, this is ready to merge already! 🚀

type-tests/tsconfig.json Show resolved Hide resolved
Copy link
Owner

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for your contribution!

@FloEdelmann FloEdelmann changed the title chore: migrate type tests to TSTyche Migrate type tests to TSTyche Feb 23, 2024
@FloEdelmann FloEdelmann merged commit 8c111e6 into FloEdelmann:main Feb 23, 2024
8 checks passed
@mrazauskas
Copy link
Contributor Author

Thanks for quick review!

It was fun to refactor the tests. Please, ping me is anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Updates or changes to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate type tests to TSTyche
2 participants