Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

feat: improve processors filtering stats #4882

Merged
merged 1 commit into from
Jul 24, 2024
Merged

feat: improve processors filtering stats #4882

merged 1 commit into from
Jul 24, 2024

Conversation

ldez
Copy link
Member

@ldez ldez commented Jul 24, 2024

The current log:

INFO [runner] Processors filtering stat (out/in): skip_files: 13145/13145, cgo: 19030/19030, exclude: 13145/13145, invalid_issue: 13145/19030, skip_dirs: 13145/13145, autogenerated_exclude: 13145/13145, nolint: 0/13145, filename_unadjuster: 19030/19030, path_prettifier: 13145/13145, identifier_marker: 13145/13145, exclude-rules: 13145/13145

The "out/in" is unexpected, I think it's better to change the order to in/out.

@ldez ldez added enhancement New feature or improvement area: processors labels Jul 24, 2024
@ldez ldez added this to the next milestone Jul 24, 2024
@ldez ldez merged commit e8300b7 into golangci:master Jul 24, 2024
16 checks passed
@ldez ldez deleted the feat/processors-filter-stats branch July 24, 2024 08:25
@ldez ldez modified the milestones: next, v1.60 Aug 14, 2024
uudashr pushed a commit to uudashr/golangci-lint that referenced this pull request Sep 18, 2024
@ldez @uudashr
feat: improve processors filtering stats (golangci#4882)
b73673c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexandear alexandear alexandear approved these changes

@bombsimon bombsimon Awaiting requested review from bombsimon

Assignees
No one assigned
Labels
area: processors enhancement New feature or improvement
Projects
None yet
Milestone
v1.60
Development

Successfully merging this pull request may close these issues.
2 participants
@ldez @alexandear