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

chore: enable goimports linter #669

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

mmorel-35
Copy link
Contributor

No description provided.

@mmorel-35 mmorel-35 requested a review from a team as a code owner February 7, 2025 10:37
@mmorel-35
Copy link
Contributor Author

/cc @crenshaw-dev

Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.46%. Comparing base (8849c3f) to head (450cbc8).
Report is 15 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #669      +/-   ##
==========================================
+ Coverage   54.26%   54.46%   +0.20%     
==========================================
  Files          64       64              
  Lines        6164     6268     +104     
==========================================
+ Hits         3345     3414      +69     
- Misses       2549     2581      +32     
- Partials      270      273       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mmorel-35 mmorel-35 force-pushed the golangci-lint/goimports branch from 2ac49c7 to d2f8707 Compare February 7, 2025 15:04
Signed-off-by: Matthieu MOREL <[email protected]>
@mmorel-35 mmorel-35 force-pushed the golangci-lint/goimports branch from d2f8707 to 450cbc8 Compare February 7, 2025 15:05
Copy link

sonarqubecloud bot commented Feb 7, 2025

@crenshaw-dev crenshaw-dev merged commit 0047207 into argoproj:master Feb 7, 2025
5 checks passed
@mmorel-35 mmorel-35 deleted the golangci-lint/goimports branch February 7, 2025 16:18
Aaron-9900 pushed a commit to Aaron-9900/gitops-engine that referenced this pull request Feb 12, 2025
Signed-off-by: Matthieu MOREL <[email protected]>
Signed-off-by: Aaron Hoffman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants