Skip to content

tools,lib: enable jsdoc/require-returns-type ESLint rule - #41130

Closed
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:require-returns-type
Closed

tools,lib: enable jsdoc/require-returns-type ESLint rule#41130
Trott wants to merge 2 commits into
nodejs:masterfrom
Trott:require-returns-type

Conversation

@Trott

@Trott Trott commented Dec 9, 2021

Copy link
Copy Markdown
Member

No description provided.

Add types to @returns JSDoc annotations where the type is missing.
@Trott Trott added the commit-queue-rebase PRs the Commit Queue should land as multiple self-contained commits. label Dec 9, 2021
@nodejs-github-bot nodejs-github-bot added lib / src Issues and PRs involving general changes in the lib/ or src/ directories. needs-ci PRs that need a full CI run. labels Dec 9, 2021
@targos

targos commented Dec 10, 2021

Copy link
Copy Markdown
Member

s/reutrns/returns/

@Trott
Trott force-pushed the require-returns-type branch from 2d21012 to 0ae51d3 Compare December 10, 2021 13:00
@Trott Trott added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 10, 2021
@Trott Trott changed the title tools,lib: enable jsdoc/require-reutrns-type ESLint rule tools,lib: enable jsdoc/require-returns-type ESLint rule Dec 10, 2021
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 10, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Trott Trott added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Dec 11, 2021
@Trott Trott added the commit-queue PRs queued for automated landing through the Commit Queue. label Dec 12, 2021
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Dec 12, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in c2ff69d...12e3c74

nodejs-github-bot pushed a commit that referenced this pull request Dec 12, 2021
Add types to @returns JSDoc annotations where the type is missing.

PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
nodejs-github-bot pushed a commit that referenced this pull request Dec 12, 2021
PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
@Trott
Trott deleted the require-returns-type branch December 12, 2021 12:43
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
Add types to @returns JSDoc annotations where the type is missing.

PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Dec 14, 2021
PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
Add types to @returns JSDoc annotations where the type is missing.

PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
Add types to @returns JSDoc annotations where the type is missing.

PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41130
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue-rebase PRs the Commit Queue should land as multiple self-contained commits. lib / src Issues and PRs involving general changes in the lib/ or src/ directories. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants