Skip to content

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