-
Notifications
You must be signed in to change notification settings - Fork 409
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
Add another stress test for SQLite #3195
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3195 +/- ##
==========================================
- Coverage 75.50% 72.08% -3.43%
==========================================
Files 396 396
Lines 22210 22210
==========================================
- Hits 16770 16009 -761
- Misses 4461 5164 +703
- Partials 979 1037 +58 see 64 files with indirect coverage changes
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
staticcheck
linter is correct there. It is hard to argue if there is a problem in FerretDB when the test itself is not correct.
Also, we don't use T{}
form when there is nothing that possibly could be initialized (there are no exported fields). See also https://dave.cheney.net/practical-go/presentations/gophercon-singapore-2019.html#_use_a_consistent_declaration_style
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Closes FerretDB#3172. Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@ferretdb.io>
Description
Closes #3172.
Readiness checklist
task all
, and it passed.@FerretDB/core
), Labels, Project and project's Sprint fields.