FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

tools: update lint-md dependencies by Trott · Pull Request #40404 · nodejs/node · GitHub

/ node Public

tools: update lint-md dependencies - #40404

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:lint-md-updates
Closed

tools: update lint-md dependencies#40404
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:lint-md-updates

Conversation

Trott commented Oct 11, 2021

Copy link
Copy Markdown
Member

Update lint-md dependencies and re-create lint-md.mjs.

Update lint-md dependencies and re-create lint-md.mjs.
nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Oct 11, 2021
Trott added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Oct 11, 2021

aduh95 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Do we still have CJS dependencies?

aduh95 commented Oct 11, 2021

Copy link
Copy Markdown
Contributor

Do we still have CJS dependencies?

So none of our direct dependencies are CJS, but some of our sub-dependencies are, so we still need the rollup plugin.

Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 13, 2021
github-actions Bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 13, 2021

Copy link
Copy Markdown
Contributor

Landed in 983b757...350a95b

github-actions Bot closed this Oct 13, 2021
nodejs-github-bot pushed a commit that referenced this pull request Oct 13, 2021
Update lint-md dependencies and re-create lint-md.mjs.

PR-URL: #40404
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
targos pushed a commit that referenced this pull request Nov 4, 2021
Update lint-md dependencies and re-create lint-md.mjs.

PR-URL: #40404
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
BethGriggs mentioned this pull request Nov 26, 2021
1 task
Trott deleted the lint-md-updates branch September 25, 2022 17:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no outstanding review comments, and a CI started. tools Issues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL