-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
Adds types support
ready |
strategy: | ||
matrix: | ||
os: [windows-latest, ubuntu-latest, macos-latest] | ||
node: [12, 14] |
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.
node: [12, 14] | |
node: [14, 15] |
This should be 14 & 15 as that's LTS/Current. Going through and updating all of these repos every time there's a new node version is going to be an enormous pain, can it be automated?
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.
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.
That issue looks like it's stalled and the linked PR is stale and also contains a bunch of irrelevant changes which will harm its chances of being merged.
I think we'll need to do something ourselves for the time being.
Adds types support