Skip to content

Conversation

@Trott
Copy link
Member

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

@nodejs-github-botnodejs-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]>
@TrottTrottforce-pushed the remove-unreachable branch from 011792d to 45cdc13CompareMarch 29, 2021 04:43
@TrottTrott merged commit 45cdc13 into nodejs:masterMar 29, 2021
@Trott
Copy link
MemberAuthor

Landed in 45cdc13

@TrottTrott 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]>
@ruyadornoruyadorno 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]>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ciPRs that need a full CI run.utilIssues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Trott@nodejs-github-bot@jasnell@cjihrig@dnlup