Skip to content

Use node v12.x in CI #7529

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

Merged
merged 2 commits into from
Aug 4, 2021
Merged

Use node v12.x in CI #7529

merged 2 commits into from
Aug 4, 2021

Conversation

giraffate
Copy link
Contributor

In #7528 (comment), remark-cli version has been updated recently and it requires to use ESM module: https://github.com/remarkjs/remark/releases/tag/14.0.0. Node version in CI was v10.24.1, it seems to be old.

changelog: none

@rust-highfive
Copy link

r? @llogiq

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 4, 2021
@llogiq
Copy link
Contributor

llogiq commented Aug 4, 2021

How did you catch that one?

@bors r+

@bors
Copy link
Contributor

bors commented Aug 4, 2021

📌 Commit f4ea50b has been approved by llogiq

@bors
Copy link
Contributor

bors commented Aug 4, 2021

⌛ Testing commit f4ea50b with merge b43eb80...

bors added a commit that referenced this pull request Aug 4, 2021
Use node v12.x in CI

In #7528 (comment), remark-cli version has been updated recently and it requires to use ESM module: https://github.com/remarkjs/remark/releases/tag/14.0.0. Node version in CI was v10.24.1, it seems to be old.

changelog: none
@bors
Copy link
Contributor

bors commented Aug 4, 2021

💔 Test failed - checks-action_remark_test

@giraffate
Copy link
Contributor Author

giraffate commented Aug 4, 2021

Syntax errors in #7528 (comment) have been resolved, but new warnings like below happen. I don't know why these happen suddenly, but I addresses it anyway.

doc/basics.md
  169:1  warning  Missing marker in block quote  no-blockquote-without-marker  remark-lint
  170:1  warning  Missing marker in block quote  no-blockquote-without-marker  remark-lint
  171:1  warning  Missing marker in block quote  no-blockquote-without-marker  remark-lint

@giraffate
Copy link
Contributor Author

How did you catch that one?

I checked error logs, npm page and changelog in remark one by one :)

@flip1995
Copy link
Member

flip1995 commented Aug 4, 2021

@bors r=flip1995,llogiq

@bors
Copy link
Contributor

bors commented Aug 4, 2021

📌 Commit 827f661 has been approved by flip1995,llogiq

@bors
Copy link
Contributor

bors commented Aug 4, 2021

⌛ Testing commit 827f661 with merge 8556910...

@bors
Copy link
Contributor

bors commented Aug 4, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: flip1995,llogiq
Pushing 8556910 to master...

@bors bors merged commit 8556910 into rust-lang:master Aug 4, 2021
@giraffate giraffate deleted the update_node_version branch August 4, 2021 10:11
joshrotenberg added a commit to joshrotenberg/rust-clippy that referenced this pull request Aug 5, 2021
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Jan 21, 2022
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Jan 27, 2022
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Apr 5, 2022
flip1995 pushed a commit to flip1995/rust-clippy that referenced this pull request Apr 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants