Skip to content

build(deps): bump golangci/golangci-lint-action from 6.5.2 to 7.0.0 #6976

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Mar 30, 2025

Bumps golangci/golangci-lint-action from 6.5.2 to 7.0.0.

Release notes

Sourced from golangci/golangci-lint-action's releases.

v7.0.0

⚠️ The GitHub Action v7 supports golangci-lint v2 only. ⚠️

What's Changed

Changes

Documentation

Dependencies

New Contributors

Full Changelog: golangci/golangci-lint-action@v6.5.2...v7.0.0

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested review from a team, skriss and sunjayBhatia and removed request for a team March 30, 2025 20:19
@dependabot dependabot bot added area/tooling Issues or PRs related to project tools and automation. release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes. labels Mar 30, 2025
@sunjayBhatia sunjayBhatia requested review from a team, rajatvig and wilsonwu and removed request for a team March 30, 2025 20:19
Copy link

The Contour project currently lacks enough contributors to adequately respond to all PRs.

This bot triages PRs according to the following rules:

  • After 14d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, the PR is closed

You can:

  • Ensure your PR is passing all CI checks. PRs that are fully green are more likely to be reviewed. If you are having trouble with CI checks, reach out to the #contour channel in the Kubernetes Slack workspace.
  • Mark this PR as fresh by commenting or pushing a commit
  • Close this PR
  • Offer to help out with triage

Please send feedback to the #contour channel in the Kubernetes Slack

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 14, 2025
@tsaarni tsaarni requested a review from a team as a code owner April 15, 2025 20:43
@tsaarni tsaarni removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 15, 2025
@tsaarni tsaarni force-pushed the dependabot/github_actions/main/golangci/golangci-lint-action-7.0.0 branch from eb8b04b to bced01f Compare April 15, 2025 20:49
@tsaarni
Copy link
Member

tsaarni commented Apr 15, 2025

golang-lint-action v7.0.0 supports golangci-lint v2 only. You can find introduction to v2 here.

After bumping the version and automatic config migration, bunch of new linting errors are produced and would need to be sorted out before this PR can be merged

@tsaarni tsaarni added the do not merge Don't merge this PR until this label is removed. label Apr 15, 2025
@sunjayBhatia sunjayBhatia added this to the 1.32.0 milestone Apr 29, 2025
@sunjayBhatia
Copy link
Member

@dependabot rebase

Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 30, 2025

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

dependabot bot and others added 3 commits April 30, 2025 15:00
Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 6.5.2 to 7.0.0.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@55c2c14...1481404)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Tero Saarni <tero.saarni@est.tech>
Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
@sunjayBhatia sunjayBhatia force-pushed the dependabot/github_actions/main/golangci/golangci-lint-action-7.0.0 branch from bced01f to c5cc784 Compare April 30, 2025 20:55
Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
Copy link

codecov bot commented May 1, 2025

Codecov Report

Attention: Patch coverage is 79.66102% with 12 lines in your changes missing coverage. Please review.

Project coverage is 80.89%. Comparing base (74c0c31) to head (9425e94).

Files with missing lines Patch % Lines
cmd/contour/serve.go 0.00% 5 Missing ⚠️
internal/dag/gatewayapi_processor.go 60.00% 0 Missing and 2 partials ⚠️
internal/debug/dot.go 0.00% 2 Missing ⚠️
internal/featuretests/v3/featuretests.go 66.66% 2 Missing ⚠️
cmd/contour/cli.go 0.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #6976      +/-   ##
==========================================
+ Coverage   80.74%   80.89%   +0.15%     
==========================================
  Files         131      131              
  Lines       19884    19850      -34     
==========================================
+ Hits        16055    16058       +3     
+ Misses       3537     3501      -36     
+ Partials      292      291       -1     
Files with missing lines Coverage Δ
cmd/contour/servecontext.go 84.91% <100.00%> (ø)
internal/annotation/annotations.go 97.14% <100.00%> (ø)
internal/dag/accessors.go 87.55% <100.00%> (ø)
internal/dag/cache.go 96.84% <100.00%> (ø)
internal/dag/ingress_processor.go 97.61% <100.00%> (ø)
internal/envoy/cluster.go 100.00% <100.00%> (ø)
internal/envoy/v3/accesslog.go 95.90% <100.00%> (+2.45%) ⬆️
internal/fixture/httpproxy.go 100.00% <100.00%> (+6.38%) ⬆️
internal/fixture/service.go 100.00% <100.00%> (ø)
internal/provisioner/objects/service/service.go 73.87% <100.00%> (ø)
... and 7 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tooling Issues or PRs related to project tools and automation. do not merge Don't merge this PR until this label is removed. release-note/none-required Marks a PR as not requiring a release note. Should only be used for very small changes.
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants