Skip to content

Conversation

Trott
Copy link
Member

@Trott Trott commented Dec 5, 2021

No description provided.

@nodejs-github-bot nodejs-github-bot added lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Dec 5, 2021
@Trott Trott added commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. request-ci Add this label to start a Jenkins CI on a PR. and removed lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. labels Dec 5, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 5, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@Trott Trott added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 7, 2021
@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 Dec 7, 2021
@nodejs-github-bot
Copy link
Collaborator

Commit Queue failed
- Loading data for nodejs/node/pull/41089
✔  Done loading data for nodejs/node/pull/41089
----------------------------------- PR info ------------------------------------
Title      tools,lib: enable jsdoc/require-returns-check lint rule (#41089)
Author     Rich Trott  (@Trott)
Branch     Trott:require-returns-check -> nodejs:master
Labels     commit-queue-rebase
Commits    2
 - lib: use consistent types in JSDoc @returns
 - tools: enable jsdoc/require-returns-check lint rule
Committers 1
 - Rich Trott 
PR-URL: https://github.com/nodejs/node/pull/41089
Reviewed-By: Michaël Zasso 
Reviewed-By: Ruben Bridgewater 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/41089
Reviewed-By: Michaël Zasso 
Reviewed-By: Ruben Bridgewater 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sun, 05 Dec 2021 04:01:42 GMT
   ✔  Approvals: 2
   ✔  - Michaël Zasso (@targos) (TSC): https://github.com/nodejs/node/pull/41089#pullrequestreview-823353142
   ✔  - Ruben Bridgewater (@BridgeAR) (TSC): https://github.com/nodejs/node/pull/41089#pullrequestreview-823371068
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-12-05T15:59:16Z: https://ci.nodejs.org/job/node-test-pull-request/41359/
- Querying data for job/node-test-pull-request/41359/
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/master up to date...
From https://github.com/nodejs/node
 * branch                  master     -> FETCH_HEAD
✔  origin/master is now up-to-date
- Downloading patch for 41089
From https://github.com/nodejs/node
 * branch                  refs/pull/41089/merge -> FETCH_HEAD
✔  Fetched commits as 3d5a7de83c97..d3881758a276
--------------------------------------------------------------------------------
Auto-merging lib/assert.js
[master 7deab87433] lib: use consistent types in JSDoc @returns
 Author: Rich Trott 
 Date: Sat Dec 4 19:59:24 2021 -0800
 4 files changed, 3 insertions(+), 4 deletions(-)
[master 71a96e9ff9] tools: enable jsdoc/require-returns-check lint rule
 Author: Rich Trott 
 Date: Sat Dec 4 19:59:37 2021 -0800
 1 file changed, 1 deletion(-)
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
lib: use consistent types in JSDoc @returns

PR-URL: #41089
Reviewed-By: Michaël Zasso targos@protonmail.com
Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

[detached HEAD f610c03f61] lib: use consistent types in JSDoc @returns
Author: Rich Trott rtrott@gmail.com
Date: Sat Dec 4 19:59:24 2021 -0800
4 files changed, 3 insertions(+), 4 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
tools: enable jsdoc/require-returns-check lint rule

PR-URL: #41089
Reviewed-By: Michaël Zasso targos@protonmail.com
Reviewed-By: Ruben Bridgewater ruben@bridgewater.de

[detached HEAD ce6a0bb7db] tools: enable jsdoc/require-returns-check lint rule
Author: Rich Trott rtrott@gmail.com
Date: Sat Dec 4 19:59:37 2021 -0800
1 file changed, 1 deletion(-)

Successfully rebased and updated refs/heads/master.

✔ f610c03f6160a5b015fd8a831ccd1f7cf9c67864
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 1:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length
✔ ce6a0bb7dbf1fc055886e5709d11730e8c8136ec
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 1:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
⚠ 0:50 Title should be <= 50 columns. title-length

The following commits are ready to be pushed to origin/master

  • f610c03f61 lib: use consistent types in JSDoc @returns
  • ce6a0bb7db tools: enable jsdoc/require-returns-check lint rule

Temporary files removed.
To finish landing:

  1. Run:
    git push origin master
  2. Post "Landed in 3d5a7de83c97...ce6a0bb7dbf1" in tools,lib: enable jsdoc/require-returns-check lint rule #41089
    gh pr comment 41089 --body "Landed in 3d5a7de83c97...ce6a0bb7dbf1"
    gh pr close 41089
    fatal: could not read Username for 'https://github.com': No such device or address
https://github.com/nodejs/node/actions/runs/1549817216

Trott added 2 commits December 7, 2021 06:23
PR-URL: nodejs#41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
PR-URL: nodejs#41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@Trott Trott force-pushed the require-returns-check branch from d388175 to e0a5877 Compare December 7, 2021 14:24
@Trott
Copy link
Member Author

Trott commented Dec 7, 2021

Landed in 3d5a7de...e0a5877

@Trott Trott merged commit e0a5877 into nodejs:master Dec 7, 2021
@Trott Trott deleted the require-returns-check branch December 7, 2021 14:24
danielleadams pushed a commit that referenced this pull request Dec 13, 2021
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Dec 13, 2021
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
PR-URL: nodejs#41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
PR-URL: nodejs#41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41089
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@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
commit-queue-failed An error occurred while landing this pull request using GitHub Actions. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants