Skip to content

Format against the shared Pint config - #10

Merged
imanimanyara merged 1 commit into
mainfrom
chore/shared-pint-config
Aug 28, 2026
Merged

imanimanyara merged 1 commit into
mainfrom
chore/shared-pint-config

Conversation

@imanimanyara

Copy link
Copy Markdown
Member

349 files. Mechanical reformat against the one pint.json now shipped by laranail/package-tools; the local copy is deleted. Suite verified before and after.

Also fixes rector here, which mapped to rector process with no --dry-run — the check script mutated the working tree.

pint.json had 24 variants across 54 files. The config now lives once in
laranail/package-tools and is consumed with --config from vendor/, so there is
no local copy left to drift.

Mechanical reformat, no behavioural change; the suite was run before and after.
Where lint was defined as a bare pint invocation it is repaired too: that FIXES
files rather than checking them, so it exited 0 unconditionally and the package
believed it was linted when it was not.
@imanimanyara
imanimanyara merged commit 98253bf into main Aug 28, 2026
@imanimanyara
imanimanyara deleted the chore/shared-pint-config branch August 28, 2026 14:22
@github-actions

Copy link
Copy Markdown

Benchmark results

Scenario Optimizations Native Laravel Optimized Speedup Δ vs base
Product import — 500 items, simple rules Wildcard, fast-check 188.5ms 3.4ms ~55x -3%
Nested order lines — 1000 orders × 5 line items Wildcard, fast-check (nested) 1292.5ms 19.2ms ~67x +1%
Event scheduling — 100 items, field-ref dates Wildcard, partial fast-check 33.0ms 1.4ms ~24x +0%
Article submission — 50 items, custom Rule objects Wildcard only 10.8ms 3.3ms ~3x +0%
Conditional import — 100 items, 47 conditional fields Wildcard, pre-evaluation 4131.8ms 54.2ms ~76x -1%
Login form — 3 fields, no wildcards Fast-check (flat) 0.2ms 0.0ms ~9x —

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.

1 participant