Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[CI:DOCS] Update dependency golangci/golangci-lint to v1.59.0 #2333

Merged
merged 1 commit into from
May 27, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 19, 2024

Mend Renovate

This PR contains the following updates:

Package Update Change
golangci/golangci-lint minor 1.58.1 -> 1.59.0

Release Notes

golangci/golangci-lint (golangci/golangci-lint)

v1.59.0

Compare Source

v1.58.2

Compare Source

  1. Updated linters
    • canonicalheader: from 1.0.6 to 1.1.1
    • gosec: from 2.19.0 to 2.20.0
    • musttag: from 0.12.1 to 0.12.2
    • nilnil: from 0.1.8 to 0.1.9
  2. Documentation
    • Improve integrations and install pages

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label May 19, 2024
Copy link

Ephemeral COPR build failed. @containers/packit-build please check.

Copy link
Member

@lsm5 lsm5 left a comment

Choose a reason for hiding this comment

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

centos-stream failing on undefined: ecdsa.HashSign. ELN failures can be ignored.

/lgtm
/approve

@lsm5
Copy link
Member

lsm5 commented May 20, 2024

whoops, validate test failed. Glad this doesn't have merge bot.

@lsm5
Copy link
Member

lsm5 commented May 20, 2024

this PR passes validate test on my rawhide env as is. Can't seem to tell why it's failing on cirrus. @cevich @mtrmac any idea whatsup here?

@mtrmac
Copy link
Contributor

mtrmac commented May 20, 2024

@lsm5 I think it’s the updated golangci-lint: it has deprecated an option. Do you have the upgraded version locally?

@mtrmac
Copy link
Contributor

mtrmac commented May 20, 2024

… except that we don’t actually set that option. Huh.

@lsm5
Copy link
Member

lsm5 commented May 20, 2024

@lsm5 I think it’s the updated golangci-lint: it has deprecated an option. Do you have the upgraded version locally?

ah nope, I was testing with a much older one installed v1.55. My bad.

@mtrmac
Copy link
Contributor

mtrmac commented May 20, 2024

Note to self: The error message was definitely added in the new 1.58.2 , but I can’t see the setting in .golangci.yml

@mtrmac
Copy link
Contributor

mtrmac commented May 20, 2024

For the record, this is a bug in golangci-lint: golangci/golangci-lint#4733 .

The linter returns exit code 0 after the warning, so we do have a way to check for lint failures without failing on this warning (that’s why c/image is reporting the same warning, but not failing).

But this warning about invalid options has helped us notice an obsolete option recently, so I’m reluctant to change hack/validate-lint.sh to ignore the warning.

Let’s wait for a week or so, to see whether golangci-lint releases an update.

@renovate renovate bot force-pushed the renovate/golangci-golangci-lint-1.x branch 3 times, most recently from aae9f0b to c8093f4 Compare May 23, 2024 13:54
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate renovate bot changed the title [CI:DOCS] Update dependency golangci/golangci-lint to v1.58.2 [CI:DOCS] Update dependency golangci/golangci-lint to v1.59.0 May 26, 2024
@renovate renovate bot force-pushed the renovate/golangci-golangci-lint-1.x branch from c8093f4 to 6ef29fd Compare May 26, 2024 19:52
@mtrmac mtrmac merged commit 49b4355 into main May 27, 2024
15 of 19 checks passed
@renovate renovate bot deleted the renovate/golangci-golangci-lint-1.x branch May 27, 2024 13:11
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Aug 26, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
dependencies Pull requests that update a dependency file locked - please file new issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants