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

tools: update lint-md-dependencies to @rollup/[email protected] #41369

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

nodejs-github-bot
Copy link
Collaborator

This is an automated update of lint-md-dependencies to @rollup/[email protected].

@nodejs-github-bot nodejs-github-bot added the tools Issues and PRs related to the tools directory. label Jan 1, 2022
@targos
Copy link
Member

targos commented Jan 3, 2022

It seems that we need to manually fix the PR to remove the first commit.

@Trott Trott force-pushed the actions/tools-update-lint-md-dependencies branch from ec26c97 to 7d508a9 Compare January 4, 2022 11:59
@Trott Trott added request-ci Add this label to start a Jenkins CI on a PR. author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Jan 4, 2022
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jan 4, 2022
@nodejs-github-bot
Copy link
Collaborator Author

@nodejs-github-bot
Copy link
Collaborator Author

@nodejs-github-bot
Copy link
Collaborator Author

@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 5, 2022
@nodejs-github-bot nodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Jan 5, 2022
@nodejs-github-bot
Copy link
Collaborator Author

Commit Queue failed
- Loading data for nodejs/node/pull/41369
✔  Done loading data for nodejs/node/pull/41369
----------------------------------- PR info ------------------------------------
Title      tools: update lint-md-dependencies to @rollup/[email protected] (#41369)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     nodejs-github-bot:actions/tools-update-lint-md-dependencies -> nodejs:master
Labels     tools, author ready
Commits    1
 - tools: update lint-md-dependencies to @rollup/[email protected]
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/41369
Reviewed-By: Michaël Zasso 
Reviewed-By: Rich Trott 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41369
Reviewed-By: Michaël Zasso 
Reviewed-By: Rich Trott 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 01 Jan 2022 00:16:59 GMT
   ✔  Approvals: 2
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/41369#pullrequestreview-842517884
   ✔  - Rich Trott (@Trott) (TSC): https://github.com/nodejs/node/pull/41369#pullrequestreview-843487435
   ✖  Last GitHub CI failed
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1657795258

@aduh95 aduh95 merged commit 1344b36 into master Jan 5, 2022
@aduh95
Copy link
Contributor

aduh95 commented Jan 5, 2022

Landed in 1344b36

danielleadams pushed a commit that referenced this pull request Feb 1, 2022
@danielleadams danielleadams mentioned this pull request Feb 1, 2022
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 pending requests for changes, and a CI started. commit-queue-failed An error occurred while landing this pull request using GitHub Actions. tools Issues and PRs related to the tools directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants