Update linter versions #308
Labels
No Label
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-s3-gw#308
Loading…
Reference in New Issue
There is no content yet.
Delete Branch "alexvanin/frostfs-s3-gw:fix/linter-per-pr"
Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?
Just wondering, what was the trigger to do this? Do we have some problems or is it time/resources of the runner?
9812b2ce9f
to5088448048
While TrueCloudLab/linters#16 is in-progress, I decided to see if
new-from-rev
approach is working, after seeing you using${{ github.event.pull_request.base.ref }}
indco.yml
.Unfortunately, runner uses fork branches.
event.pull_request.base.ref
is resolved toorigin/master
and my forkedmaster
branch is a bit outdated. Therefore linter checks multiple commits instead of one in this PR.2ea0265160
tob5bcdbe087
b5bcdbe087
to563c1d9bd7
With the new linter release, everything works just fine, so I dropped all other changes in this PR.
WIP: Use new-from-rev flag for linter checkto Use new-from-rev flag for linter checkUse new-from-rev flag for linter checkto Update linter versions