Sitelet https://github.com/transparency-dev/formats/pull/183
Skip to content

Bump golangci lint - #183

Merged
AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:bump_golangci-lint
Apr 3, 2025
Merged

AlCutter merged 3 commits into
transparency-dev:mainfrom
AlCutter:bump_golangci-lint

Conversation

@AlCutter

Copy link
Copy Markdown
Collaborator

This PR bumps golangci-lint@v2 and migrates the config file.

Should unblock #182.

@AlCutter
AlCutter requested a review from a team as a code owner March 31, 2025 11:21
@AlCutter
AlCutter requested a review from roger2hk March 31, 2025 11:21
@AlCutter AlCutter added the dependencies Pull requests that update a dependency file label Mar 31, 2025
@codecov-commenter

codecov-commenter commented Mar 31, 2025 •

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.00%. Comparing base (fa00c16) to head (01aaea6).
Report is 101 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
- Coverage   82.57%   75.00%   -7.58%     
==========================================
  Files           5        6       +1     
  Lines         241      436     +195     
==========================================
+ Hits          199      327     +128     
- Misses         30       78      +48     
- Partials       12       31      +19     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Comment thread .golangci.yaml Outdated
Comment on lines +12 to +24
exclusions:
generated: lax
paths:
- third_party$
- builtin$
- examples$
formatters:
exclusions:
generated: lax
paths:
- third_party$
- builtin$
- examples$

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These paths don't exist in this repo.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I imagine it's just being explicit about defaults (or attempting to preserve defaults from earlier version?)

@AlCutter

AlCutter commented Apr 3, 2025

Copy link
Copy Markdown
Collaborator Author

@roger2hk need another approve due to pushing the commit to remove the paths :/

@AlCutter
AlCutter merged commit 2023689 into transparency-dev:main Apr 3, 2025
@AlCutter
AlCutter deleted the bump_golangci-lint branch April 3, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants