Fix pin versions nyc version for testing on node 8 & 9 by carpasse · Pull Request #31 · jshttp/statuses (original) (raw)

Comment on lines +107 to +108

- name: Node.js 17.x
node-version: "17.2"

Choose a reason for hiding this comment

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

Can we move this to a separate PR? Or include up to 21 in this one? I will suggest to keep the PR the most atomic possible and avoid add new Node.js versions in this fix.

Choose a reason for hiding this comment

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

I will move it to a separate PR

Choose a reason for hiding this comment

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

removed

Choose a reason for hiding this comment

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

PR adding new versions #32