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

Remove support for Node.js 12 in CLI #1007

Merged
merged 2 commits into from
Jun 24, 2022
Merged

Remove support for Node.js 12 in CLI #1007

merged 2 commits into from
Jun 24, 2022

Conversation

remcohaszing
Copy link
Member

Initial checklist

  • I read the support docs
  • I read the contributing guide
  • I agree to follow the code of conduct
  • I searched issues and couldn’t find anything (or linked relevant results below)
  • If applicable, I’ve added docs and tests

Description of changes

This is a preparation for the update of unified-args in remark-cli. This also means none of the other packages in the mono repo are tested against Node 12 anymore.

This is a preparation for the update of unified-args.
@github-actions github-actions bot added the 👋 phase/new Post is being triaged automatically label Jun 23, 2022
@github-actions github-actions bot added 🤞 phase/open Post is being triaged manually and removed 👋 phase/new Post is being triaged automatically labels Jun 23, 2022
@remcohaszing remcohaszing added the 🧑 semver/major This is a change label Jun 23, 2022
@wooorm
Copy link
Member

wooorm commented Jun 23, 2022

Can we do it for the CLI (and CI) only, so as not to require a major for remark/remark-parse/remark-stringify?

@remcohaszing
Copy link
Member Author

As in, revert the changes in the other readmes?

@wooorm
Copy link
Member

wooorm commented Jun 23, 2022

Yup!

Node.js 12 support is only dropped for `remark-cli`.
@codecov-commenter

This comment was marked as resolved.

@wooorm wooorm changed the title Drop support for Node.js 12 Remove support for Node.js 12 in CLI Jun 24, 2022
@wooorm wooorm merged commit 56eb9ed into remarkjs:main Jun 24, 2022
@wooorm wooorm added the 💪 phase/solved Post is done label Jun 24, 2022
@github-actions

This comment has been minimized.

@github-actions github-actions bot removed the 🤞 phase/open Post is being triaged manually label Jun 24, 2022
bors added a commit to rust-lang/rust-clippy that referenced this pull request Jun 26, 2022
…ishearth

update node.js version in `remark.yml`

Optional chain (`?.`) is available in `node v14`, but node version in CI is `node v12`, so CI is failed now.

ref:
https://github.com/rust-lang/rust-clippy/runs/7059529735?check_suite_focus=true

optional chain:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Optional_chaining#browser_compatibility

Corresponding PR (maybe)
remarkjs/remark#1007

changelog: None
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 phase/solved Post is done 🧑 semver/major This is a change
Development

Successfully merging this pull request may close these issues.

4 participants