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.

Sign up for GitHub

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

Jump to bottom

tools: use Set instead of { [key]: true } object #41675

Merged
merged 1 commit into from
Jan 24, 2022
Merged

tools: use Set instead of { [key]: true } object #41675

merged 1 commit into from
Jan 24, 2022

Conversation

tniessen
Copy link
Member

A Set has clearer semantics than an object whose values are always true.

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Jan 24, 2022
@benjamingr benjamingr added the fast-track PRs that do not need to wait for 48 hours to land. label Jan 24, 2022
@github-actions
Copy link
Contributor

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

@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 24, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jan 24, 2022
@nodejs-github-bot nodejs-github-bot merged commit 400b7c2 into nodejs:master Jan 24, 2022
@nodejs-github-bot
Copy link
Collaborator

Landed in 400b7c2

BethGriggs pushed a commit that referenced this pull request Jan 25, 2022
@tniessen @BethGriggs
tools: use Set instead of { [key]: true } object
5fe7cfa 
PR-URL: #41675
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
tniessen added a commit to tniessen/node that referenced this pull request Jan 25, 2022
@tniessen
tools: use Set instead of { [key]: true } object
442217e 
Refs: nodejs#41675
@tniessen tniessen mentioned this pull request Jan 25, 2022
Merged
nodejs-github-bot pushed a commit that referenced this pull request Jan 28, 2022
@tniessen
tools: use Set instead of { [key]: true } object
334fb38 
Refs: #41675

PR-URL: #41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
@tniessen @Linkgoron
tools: use Set instead of { [key]: true } object
791365f 
PR-URL: nodejs#41675
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Linkgoron pushed a commit to Linkgoron/node that referenced this pull request Jan 31, 2022
@tniessen @Linkgoron
tools: use Set instead of { [key]: true } object
be9b554 
Refs: nodejs#41675

PR-URL: nodejs#41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
ruyadorno pushed a commit that referenced this pull request Feb 8, 2022
@tniessen @ruyadorno
tools: use Set instead of { [key]: true } object
3190e36 
Refs: #41675

PR-URL: #41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@ruyadorno ruyadorno mentioned this pull request Feb 8, 2022
Merged
danielleadams pushed a commit that referenced this pull request Mar 2, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
55341ee 
PR-URL: #41675
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Mar 2, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
799f850 
Refs: #41675

PR-URL: #41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@danielleadams danielleadams mentioned this pull request Mar 3, 2022
Merged
danielleadams pushed a commit that referenced this pull request Mar 3, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
543f168 
PR-URL: #41675
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Mar 3, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
3918467 
Refs: #41675

PR-URL: #41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
danielleadams pushed a commit that referenced this pull request Mar 14, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
1cbdc98 
PR-URL: #41675
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
danielleadams pushed a commit that referenced this pull request Mar 14, 2022
@tniessen @danielleadams
tools: use Set instead of { [key]: true } object
9d23a27 
Refs: #41675

PR-URL: #41695
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Mestery <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aduh95 aduh95 aduh95 approved these changes

@targos targos targos approved these changes

@benjamingr benjamingr benjamingr approved these changes

Assignees
No one assigned
Labels
doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land. tools Issues and PRs related to the tools directory.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tniessen @nodejs-github-bot @benjamingr @targos @aduh95