lasiar / canonicalheader

Golang linter checking the canonicality of the http header
MIT License
6 stars 1 forks source link

Bump linter #37

Closed lasiar closed 3 weeks ago

ccoVeille commented 3 weeks ago

I'm curious

Wouldn't dependabot have bumped it?

I'm unsure as it's a simple parameter of golangci-lint action.

Or is it because you didn't want to wait for the weekly dependabot to be triggered ?

Thanks

lasiar commented 3 weeks ago

Hi, @ccoVeille!

Yes, this is just parameter, dependabot cannot work with them

ccoVeille commented 3 weeks ago

Hi, @ccoVeille!

Yes, this is just parameter, dependabot cannot work with them

Thanks for confirmation

ccoVeille commented 3 weeks ago

Just checked code and documentation

You could use latest https://github.com/golangci/golangci-lint-action

ccoVeille commented 3 weeks ago

The same way could use stable+ oldstable for go-version with setup-go

https://github.com/marketplace/actions/setup-go-environment#using-stableoldstable-aliases

ccoVeille commented 3 weeks ago

I think renovate GHA not would be able to catch this kind of upgrade, you can have a look here

https://github.com/deepmap/oapi-codegen/pull/1603

Did you ever try it?

But using latest as I suggested in a previous message could also be OK

Any thoughts about it ? @lasiar