Skip to content

tools: use Object.hasOwn() in alljson.mjs - #41306

Merged
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
Trott:alljson-hasown
Dec 26, 2021
Merged

tools: use Object.hasOwn() in alljson.mjs#41306
nodejs-github-bot merged 1 commit into
nodejs:masterfrom
Trott:alljson-hasown

Conversation

@Trott

Copy link
Copy Markdown
Member

Replace hasOwnProperty() with Object.hasOwn().

Replace hasOwnProperty() with Object.hasOwn().
@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Dec 23, 2021
@aduh95aduh95 added dont-land-on-v12.x author ready PRs that have at least one approval, no pending requests for changes, and a CI started. labels Dec 24, 2021
@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Dec 24, 2021
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Dec 24, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@TrottTrott added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 26, 2021
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 26, 2021
@nodejs-github-bot
nodejs-github-bot merged commit b330ab8 into nodejs:masterDec 26, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in b330ab8

targos pushed a commit that referenced this pull request Jan 14, 2022
Replace hasOwnProperty() with Object.hasOwn().
PR-URL: #41306
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@targostargos mentioned this pull request Jan 16, 2022
danielleadams pushed a commit that referenced this pull request Jan 31, 2022
Replace hasOwnProperty() with Object.hasOwn().
PR-URL: #41306
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this pull request Feb 1, 2022
Replace hasOwnProperty() with Object.hasOwn().
PR-URL: #41306
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: James M Snell <jasnell@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.docIssues and PRs related to the documentations.toolsIssues and PRs related to the tools directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@Trott@nodejs-github-bot@jasnell@tniessen@aduh95@VoltrexKeyva