Skip to content

tools: enable prefer-object-has-own lint rule - #41245

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
Trott:prefer-has-own
Dec 21, 2021
Merged

tools: enable prefer-object-has-own lint rule#41245
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
Trott:prefer-has-own

Conversation

@Trott

@TrottTrott commented Dec 19, 2021

Copy link
Copy Markdown
Member

First commit is from #41229.

Second commit enables the lint rule. Ref: https://eslint.org/docs/rules/prefer-object-has-own

@TrottTrott added fast-track PRs that do not need to wait for 48 hours to land. dont-land-on-v12.x labels Dec 19, 2021
@github-actions

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @Trott. Please 👍 to approve.

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. labels Dec 19, 2021
@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 19, 2021
@nodejs-github-bot

This comment has been minimized.

@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 20, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 20, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 20, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@TrottTrott added commit-queue Add this label to land a pull request using GitHub Actions. and removed fast-track PRs that do not need to wait for 48 hours to land. labels Dec 21, 2021
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 21, 2021
@nodejs-github-bot
nodejs-github-bot merged commit 340998e into nodejs:masterDec 21, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 340998e

@Trott
Trott deleted the prefer-has-own branch December 21, 2021 18:17
targos pushed a commit that referenced this pull request Jan 14, 2022
PR-URL: #41245
Refs: https://eslint.org/docs/rules/prefer-object-has-own
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@targostargos mentioned this pull request Jan 16, 2022
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
PR-URL: #41245
Refs: https://eslint.org/docs/rules/prefer-object-has-own
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@danielleadamsdanielleadams mentioned this pull request Feb 1, 2022
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.needs-ciPRs that need a full CI run.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants

@Trott@nodejs-github-bot@jasnell@lpinca@cjihrig@tniessen@BridgeAR@aduh95@danielleadams