Skip to content

⬆ Bump golangci/golangci-lint-action from 7 to 9#322

Merged
openshift-merge-bot[bot] merged 1 commit into
oadp-devfrom
dependabot/github_actions/golangci/golangci-lint-action-9
Jun 3, 2026
Merged

⬆ Bump golangci/golangci-lint-action from 7 to 9#322
openshift-merge-bot[bot] merged 1 commit into
oadp-devfrom
dependabot/github_actions/golangci/golangci-lint-action-9

Conversation

@dependabot
Copy link
Copy Markdown
Contributor

@dependabot dependabot Bot commented on behalf of github Nov 9, 2025

Bumps golangci/golangci-lint-action from 7 to 9.

Release notes

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

v9.0.0

In the scope of this release, we change Nodejs runtime from node20 to node24 (https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/).

What's Changed

Changes

Full Changelog: golangci/golangci-lint-action@v8.0.0...v9.0.0

v8.0.0

Requires golangci-lint version >= v2.1.0

What's Changed

Changes

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

v7.0.1

What's Changed

Documentation

Dependencies

New Contributors

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

Commits
  • 82606bf chore: prepare release v9.2.1
  • 97c8387 chore: improve workflows (#1394)
  • 28d0a19 build(deps): bump the dependencies group across 1 directory with 2 updates
  • 633fbc7 build(deps): bump github/codeql-action from 4.35.3 to 4.35.4 (#1391)
  • 59f43e2 build(deps): bump github/codeql-action from 4.35.2 to 4.35.3 (#1389)
  • 9eb174e build(deps): bump fast-xml-builder from 1.1.5 to 1.2.0 (#1386)
  • 4f52504 build(deps): bump github/codeql-action from 4 to 4.35.2 (#1384)
  • 6f87dfd docs: update examples
  • c9500d7 chore: improve workflows
  • 03b1faa chore: improve issue templates
  • Additional commits viewable in compare view

@dependabot dependabot Bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Nov 9, 2025
@oadp-snyk
Copy link
Copy Markdown

oadp-snyk commented Nov 9, 2025

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Nov 9, 2025

Hi @dependabot[bot]. Thanks for your PR.

I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Copy link
Copy Markdown
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

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

/ok-to-test
/lgtm

@petr-muller
Copy link
Copy Markdown

/hold

PR is unmergable. Do the Tide merge requirements match the GitHub settings for the repo? At least 2 approving reviews are required by reviewers with write access.

https://prow.ci.openshift.org/tide-history?repo=migtools%2Foadp-non-admin&pull=322&state=failure

@kaovilai
Copy link
Copy Markdown
Member

kaovilai commented Jun 2, 2026

@petr-muller yes it is known issue. PR is not mergeable.. need one more review.

Do you prefer lgtm NOT be added while kubernetes-sigs/prow#673 is unsolved? Or is it consuming CPU cycles too much?

@kaovilai
Copy link
Copy Markdown
Member

kaovilai commented Jun 2, 2026

second reviewer please unhold

@kaovilai
Copy link
Copy Markdown
Member

kaovilai commented Jun 2, 2026

@dependabot rebase

Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 7 to 9.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v7...v9)

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

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot force-pushed the dependabot/github_actions/golangci/golangci-lint-action-9 branch from d06e442 to 36510e3 Compare June 2, 2026 19:29
@openshift-ci openshift-ci Bot removed the lgtm label Jun 2, 2026
@petr-muller
Copy link
Copy Markdown

Do you prefer lgtm NOT be added while kubernetes-sigs/prow#673 is unsolved? Or is it consuming CPU cycles too much?

I don't care that much, just noticed this PR is stuck in this state since Nov 2025 and thought it is unlikely to get a second review so I just poked

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 3, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dependabot[bot], kaovilai, weshayutin

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kaovilai
Copy link
Copy Markdown
Member

kaovilai commented Jun 3, 2026

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 3, 2026
@kaovilai
Copy link
Copy Markdown
Member

kaovilai commented Jun 3, 2026

/unhold

@openshift-merge-bot openshift-merge-bot Bot merged commit 58c8b91 into oadp-dev Jun 3, 2026
3 checks passed
@openshift-merge-bot openshift-merge-bot Bot deleted the dependabot/github_actions/golangci/golangci-lint-action-9 branch June 3, 2026 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code lgtm ok-to-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants