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.

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

test: remove test-sqlite-statement-sync flaky designation #56051

Merged

Conversation

lpinca
Copy link
Member

@lpinca lpinca commented Nov 28, 2024

The issue is likely the same as other tests that time out.

Refs: #54918
Refs: #54802

@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 Nov 28, 2024
The issue is likely the same as other tests that time out.

Refs: nodejs#54918
Refs: nodejs#54802
@lpinca lpinca force-pushed the remove/flaky-test-sqlite-statement-sync branch from 421783b to 68be957 Compare November 28, 2024 10:24
Copy link

codecov bot commented Nov 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.96%. Comparing base (24a8662) to head (68be957).
Report is 135 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #56051      +/-   ##
==========================================
- Coverage   87.97%   87.96%   -0.01%     
==========================================
  Files         656      656              
  Lines      188383   188383              
  Branches    35973    35971       -2     
==========================================
- Hits       165732   165716      -16     
- Misses      15821    15837      +16     
  Partials     6830     6830              

see 35 files with indirect coverage changes

@lpinca lpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 28, 2024
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 28, 2024
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@lpinca
Copy link
Member Author

lpinca commented Dec 7, 2024

@targos would you mind re-approving so that this can land? Thank you.

@lpinca
Copy link
Member Author

lpinca commented Dec 17, 2024

@nodejs/testing

@lpinca lpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 17, 2024
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 17, 2024
@nodejs-github-bot nodejs-github-bot merged commit 6536978 into nodejs:main Dec 17, 2024
59 checks passed
@nodejs-github-bot
Copy link
Collaborator

Landed in 6536978

@lpinca lpinca deleted the remove/flaky-test-sqlite-statement-sync branch December 17, 2024 13:07
aduh95 pushed a commit that referenced this pull request Dec 18, 2024
The issue is likely the same as other tests that time out.

Refs: #54918
Refs: #54802
PR-URL: #56051
Reviewed-By: Michaël Zasso <[email protected]>
aduh95 pushed a commit that referenced this pull request Dec 18, 2024
The issue is likely the same as other tests that time out.

Refs: #54918
Refs: #54802
PR-URL: #56051
Reviewed-By: Michaël Zasso <[email protected]>
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. test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants