Skip to content

Linter CLI: Implement a Linter Worker to parallelize file processing#1371

Merged
marcoroth merged 1 commit intomainfrom
linter-worker
Mar 12, 2026
Merged

Linter CLI: Implement a Linter Worker to parallelize file processing#1371
marcoroth merged 1 commit intomainfrom
linter-worker

Conversation

@marcoroth
Copy link
Owner

@marcoroth marcoroth commented Mar 12, 2026

This pull request implements a linter worker to speed up the file processing of the linter CLI.

On a quick initial test, it seems to be quite significant using the default (auto) option:

App 1

Before

 Summary:
  Checked      421 files
  Files        24 with offenses | 397 clean (421 total)
  Offenses     23 errors | 33 warnings | 1 info (57 offenses across 24 files)
  Fixable      57 offenses | 25 autocorrectable using `--fix`
  Start at     01:58:43
  Duration     2958ms (71 rules)

After

 Summary:
  Checked      421 files
  Files        24 with offenses | 397 clean (421 total)
  Offenses     23 errors | 33 warnings | 1 info (57 offenses across 24 files)
  Fixable      57 offenses | 25 autocorrectable using `--fix`
  Start at     01:57:40
  Duration     1264ms (71 rules)

App 2

Before

 Summary:
  Checked      1384 files
  Files        564 with offenses | 820 clean (1384 total)
  Offenses     1998 errors | 282 warnings (2280 offenses across 564 files)
  Fixable      2280 offenses | 591 autocorrectable using `--fix`
  Start at     01:59:52
  Duration     7292ms (70 rules)

After

 Summary:
  Checked      1384 files
  Files        564 with offenses | 820 clean (1384 total)
  Offenses     1998 errors | 282 warnings (2280 offenses across 564 files)
  Fixable      2280 offenses | 591 autocorrectable using `--fix`
  Start at     01:59:20
  Duration     2427ms (70 rules)

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 12, 2026

npx https://pkg.pr.new/@herb-tools/formatter@1371
npx https://pkg.pr.new/@herb-tools/language-server@1371
npx https://pkg.pr.new/@herb-tools/linter@1371

commit: e42d60f

@github-actions
Copy link

github-actions bot commented Mar 12, 2026

🌿 Interactive Playground and Documentation Preview

A preview deployment has been built for this pull request. Try out the changes live in the interactive playground:


🌱 Grown from commit e42d60f


✅ Preview deployment has been cleaned up.

@marcoroth marcoroth merged commit a0ec12c into main Mar 12, 2026
20 checks passed
@marcoroth marcoroth deleted the linter-worker branch March 12, 2026 01:05
@marcoroth marcoroth added this to the v1.0.0 milestone Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant