Skip to content

Conversation

aduh95
Copy link
Contributor

@aduh95 aduh95 commented Apr 28, 2024

IIUC #52674 is not yet ready to land, so let's mark the test as flaky in the mean time.

Refs: #52630

@aduh95 aduh95 added the fast-track PRs that do not need to wait for 48 hours to land. label Apr 28, 2024
Copy link
Contributor

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

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Apr 28, 2024
@nodejs-github-bot
Copy link
Collaborator

@aduh95 aduh95 force-pushed the test-error-serdes-flaky branch from 17707a1 to 282d078 Compare April 28, 2024 20:52
@aduh95 aduh95 changed the title testL make test-error-serdes as flaky test: make test-error-serdes as flaky Apr 28, 2024
@nodejs-github-bot
Copy link
Collaborator

@aduh95 aduh95 merged commit 7c3dce0 into nodejs:main Apr 28, 2024
@aduh95
Copy link
Contributor Author

aduh95 commented Apr 28, 2024

Landed in 7c3dce0

@aduh95 aduh95 deleted the test-error-serdes-flaky branch April 28, 2024 23:10
aduh95 added a commit that referenced this pull request Apr 29, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
marco-ippolito pushed a commit that referenced this pull request May 2, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
marco-ippolito pushed a commit that referenced this pull request May 3, 2024
PR-URL: #52739
Refs: #52630
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
eliphazbouye pushed a commit to eliphazbouye/node that referenced this pull request Jun 20, 2024
PR-URL: nodejs#52739
Refs: nodejs#52630
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
bmeck pushed a commit to bmeck/node that referenced this pull request Jun 22, 2024
PR-URL: nodejs#52739
Refs: nodejs#52630
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fast-track PRs that do not need to wait for 48 hours to land. needs-ci PRs that need a full CI run. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants