Skip to content

Conversation

Trott
Copy link
Member

@Trott Trott commented Mar 27, 2021

Convert invariant from if statement to an assertion. The condition is
believed to be impossible to trigger.

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. labels Mar 27, 2021
@nodejs-github-bot
Copy link
Collaborator

Convert invariant from if statement to an assertion. The condition is
believed to be impossible to trigger.

PR-URL: nodejs#37941
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@Trott Trott force-pushed the remove-unreachable branch from 011792d to 45cdc13 Compare March 29, 2021 04:43
@Trott Trott merged commit 45cdc13 into nodejs:master Mar 29, 2021
@Trott
Copy link
Member Author

Trott commented Mar 29, 2021

Landed in 45cdc13

@Trott Trott deleted the remove-unreachable branch March 29, 2021 04:44
ruyadorno pushed a commit that referenced this pull request Mar 29, 2021
Convert invariant from if statement to an assertion. The condition is
believed to be impossible to trigger.

PR-URL: #37941
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
ruyadorno pushed a commit that referenced this pull request Mar 30, 2021
Convert invariant from if statement to an assertion. The condition is
believed to be impossible to trigger.

PR-URL: #37941
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@ruyadorno ruyadorno mentioned this pull request Mar 30, 2021
targos pushed a commit that referenced this pull request May 1, 2021
Convert invariant from if statement to an assertion. The condition is
believed to be impossible to trigger.

PR-URL: #37941
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@danielleadams danielleadams mentioned this pull request May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants