Skip to content

Introduce auto-fixer capabilities - #35

Merged
jordikroon merged 17 commits into
php:mainfrom
NickSdot:stack-04-fixers
Jul 25, 2026
Merged

Introduce auto-fixer capabilities#35
jordikroon merged 17 commits into
php:mainfrom
NickSdot:stack-04-fixers

Conversation

@NickSdot

@NickSdotNickSdot commented Jul 20, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Adds auto-fixer. 🥳

Depends on #29, #30, #31, #32, #33, and #34.
This PRs diff will shrink as preceding PRs are merged and this branch is rebased.

Diff against previous branch: NickSdot:stack-03-report-paths...NickSdot:stack-04-fixers (different to diff in here only until the previous branch was merged)

This was referenced Jul 21, 2026
Closed
Merged
This was referenced Jul 21, 2026
Merged
Merged
Merged
@NickSdot
NickSdotforce-pushed the stack-04-fixers branch 3 times, most recently from 3071a50 to e815115CompareJuly 21, 2026 08:44
@NickSdotNickSdot mentioned this pull request Jul 21, 2026
Merged
@NickSdot
NickSdotforce-pushed the stack-04-fixers branch 2 times, most recently from 02334f6 to bda2c53CompareJuly 22, 2026 07:39
@NickSdotNickSdot mentioned this pull request Jul 22, 2026
Merged
@KentarouTakedaKentarouTakeda mentioned this pull request Jul 24, 2026
Closed
@NickSdot
NickSdotforce-pushed the stack-04-fixers branch 4 times, most recently from 69993a5 to cbb7d2fCompareJuly 25, 2026 09:27
@NickSdotNickSdot mentioned this pull request Jul 25, 2026
Merged
NickSdot
NickSdot commented Jul 25, 2026
View reviewed changes
Comment threadsrc/Fix/Fixer/WhitespaceFixer.php Outdated
@NickSdot

NickSdot commented Jul 25, 2026

Copy link
Copy Markdown
ContributorAuthor

Rebased and ready for review.

@jordikroon
jordikroon marked this pull request as ready for review July 25, 2026 09:44
NickSdot
NickSdot commented Jul 25, 2026
View reviewed changes
Comment threadsrc/Fix/Fixer/MixedIndentationFixer.php Outdated
NickSdot
NickSdot commented Jul 25, 2026
View reviewed changes
Comment threadsrc/Runner/RunCoordinator.php
NickSdot
NickSdot commented Jul 25, 2026
View reviewed changes
Comment threadsrc/Runner/XmlFileProcessor.php
jordikroon
jordikroon requested changes Jul 25, 2026
View reviewed changes
Comment threadsrc/Fix/Fixer/MixedIndentationFixer.php Outdated
Comment threadsrc/Fix/FixApplier.php Outdated
Comment threadsrc/Path/DiffPathLoader.php
Comment threadsrc/Path/EntityResolver.php
Comment threadtests/Unit/ApplicationTest.php
Comment threadsrc/Fix/Fixer/WhitespaceFixer.php Outdated
Comment threadsrc/Runner/RunScope.php
Comment threadsrc/Runner/RunScope.php Outdated
Comment threadsrc/Runner/XmlFileProcessor.php
Comment threadsrc/Sniff/AttributeOrderSniff.php Outdated
@NickSdot

NickSdot commented Jul 25, 2026

Copy link
Copy Markdown
ContributorAuthor

All done. Ready for review again.

@NickSdot
NickSdot requested a review from jordikroonJuly 25, 2026 16:17
@jordikroon

jordikroon commented Jul 25, 2026

Copy link
Copy Markdown
Member

Okay let's make this happen! 🔥

@jordikroon
jordikroon merged commit cee75f1 into php:mainJul 25, 2026
3 checks passed
jordikroon added a commit to php/doc-en that referenced this pull request Jul 25, 2026
@jordikroon
Split whitespace sniffers introduced inphp/docbook-cs#35
33b09ed
This was referenced Jul 25, 2026
Merged
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jordikroonjordikroonjordikroon approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@NickSdot@jordikroon