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

Reformat code after updating golangci linter #4099

Closed
wants to merge 1 commit into from

Conversation

rumyantseva
Copy link
Contributor

@rumyantseva rumyantseva commented Feb 19, 2024

Description

Fix everything the linter complains about.

Readiness checklist

  • I added/updated unit tests (and they pass).
  • I added/updated integration/compatibility tests (and they pass).
  • I added/updated comments and checked rendering.
  • I made spot refactorings.
  • I updated user documentation.
  • I ran task all, and it passed.
  • I ensured that PR title is good enough for the changelog.
  • (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
  • I marked all done items in this checklist.

@rumyantseva rumyantseva added the code/chore Code maintenance improvements label Feb 19, 2024
@rumyantseva rumyantseva self-assigned this Feb 19, 2024
@rumyantseva rumyantseva added this to the Next milestone Feb 19, 2024
Copy link

codecov bot commented Feb 19, 2024

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (74d6428) 75.85% compared to head (ee26a4c) 74.74%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4099      +/-   ##
==========================================
- Coverage   75.85%   74.74%   -1.11%     
==========================================
  Files         347      347              
  Lines       22195    22194       -1     
==========================================
- Hits        16836    16589     -247     
- Misses       4031     4319     +288     
+ Partials     1328     1286      -42     
Files Coverage Δ
internal/wire/op_msg.go 60.75% <0.00%> (-0.21%) ⬇️

... and 11 files with indirect coverage changes

Flag Coverage Δ
filter-true 66.28% <0.00%> (-1.27%) ⬇️
hana-1 ?
integration 66.28% <0.00%> (-1.27%) ⬇️
mongodb-1 4.72% <0.00%> (+<0.01%) ⬆️
postgresql-1 48.95% <0.00%> (-0.08%) ⬇️
postgresql-2 46.88% <0.00%> (-0.03%) ⬇️
postgresql-3 47.22% <0.00%> (-0.01%) ⬇️
sqlite-1 48.32% <0.00%> (+0.07%) ⬆️
sqlite-2 45.95% <0.00%> (-0.01%) ⬇️
sqlite-3 46.39% <0.00%> (-0.01%) ⬇️
unit 32.87% <0.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@rumyantseva rumyantseva marked this pull request as ready for review February 19, 2024 20:49
@rumyantseva rumyantseva requested a review from a team as a code owner February 19, 2024 20:49
@rumyantseva rumyantseva enabled auto-merge (squash) February 19, 2024 20:49
@rumyantseva rumyantseva requested review from a team, henvic and noisersup February 19, 2024 20:49
Copy link
Member

@AlekSi AlekSi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As you can see, the linter does not complain on CI: https://github.com/FerretDB/FerretDB/actions/runs/7966637338/job/21748215090
That's because origin/main is up-to-date there. If you see linter errors in this file locally, that's because your branches are not in sync. Please understand how new-from-rev option works.

This code is being rewritten in my branch anyway.

@rumyantseva
Copy link
Contributor Author

rumyantseva commented Feb 20, 2024

@AlekSi no, I don't see any problems locally. Locally everything passes. But in this PR #4096, the linter fails on files lines that I haven't touched.

@rumyantseva rumyantseva requested a review from AlekSi February 20, 2024 07:23
auto-merge was automatically disabled February 20, 2024 07:27

Pull request was closed

@AlekSi AlekSi removed this from the Next milestone Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code/chore Code maintenance improvements
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants