-
Notifications
You must be signed in to change notification settings - Fork 5
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
chore: set TypeScript version range to >=4.2 #75
Conversation
24c3066
to
3046da3
Compare
3046da3
to
0d542e3
Compare
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.
Thanks @JoshuaKGoldberg!
name: Format | ||
|
||
on: | ||
pull_request: ~ |
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.
TIL that ~
means null
in YAML. Perhaps just write it as null
for clarity?
Is this saying that we don't want to run format checks on PR branches? Why would that be?
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.
Heh I went the opposite conclusion in https://github.com/JoshuaKGoldberg/template-typescript-node-package - that it implies "run on all of them", which null
feels ... the opposite to me?
It means we run format checks on:
- All pull request commits
- Commits to
main
🎉 This is included in version v0.3.0 🎉 The release is available on: Cheers! 📦🚀 |
PR Checklist
status: accepting prs
Overview
Sets the minimum tested TypeScript version to 4.2, with a matrix of tests across versions up through
5.0
andlatest
.Borrows some GHA configuration from https://github.com/JoshuaKGoldberg/template-typescript-node-package.