Skip to content

ci: address zizmor findings - #3871

Merged
ArthurSens merged 1 commit into
prometheus:masterfrom
TheMeier:zizmor
Oct 29, 2026
Merged

ArthurSens merged 1 commit into
prometheus:masterfrom
TheMeier:zizmor

Conversation

@TheMeier

Copy link
Copy Markdown
Contributor

No description provided.

@TheMeier

Copy link
Copy Markdown
Contributor Author

regards #3867

@TheMeier

Copy link
Copy Markdown
Contributor Author

@SuperQ HTH, If not just close it and sorry for the noise

Comment thread .github/workflows/ci.yml
- uses: actions/setup-go@b7ad1dad31e06c5925ef5d2fc7ad053ef454303e # v7.0.0
with:
go-version: 1.27.x
cache: false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm pretty sure this one is a false-positive since there is no publish in this flow.

CC @ArthurSens

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

yep, we probably want to ignore this one

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm merging even if this one isn't solved because the next repo_sync will fix this

uses: actions/setup-go@b7ad1dad31e06c5925ef5d2fc7ad053ef454303e # v7.0.0
with:
go-version: 1.27.x
cache: false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same here, I think we added a nolint in the upstream prometheus version.

@SuperQ

SuperQ commented Oct 26, 2026

Copy link
Copy Markdown
Member

Yea, this is helpful. It's just tedious. :)

Comment thread .github/workflows/ci.yml Resolved
@TheMeier
TheMeier force-pushed the zizmor branch 2 times, most recently from b9e1ab8 to 8b3b7b5 Compare October 27, 2026 03:37
Comment thread .github/workflows/ci.yml Resolved
Signed-off-by: Christoph Maser <christoph.maser+github@gmail.com>
@ArthurSens
ArthurSens merged commit 3af7d9f into prometheus:master Oct 29, 2026
17 checks passed
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.

3 participants