Skip to content

fix: lint autofix — 97 errors → 19, 249 warnings → 6#822

Merged
chubes4 merged 1 commit intomainfrom
fix/lint-autofix
Mar 16, 2026
Merged

fix: lint autofix — 97 errors → 19, 249 warnings → 6#822
chubes4 merged 1 commit intomainfrom
fix/lint-autofix

Conversation

@chubes4
Copy link
Member

@chubes4 chubes4 commented Mar 16, 2026

Summary

  • Applied homeboy lint data-machine --fix across 46 files
  • Before: 97 errors, 249 warnings (308 auto-fixable)
  • After: 19 errors, 6 warnings
  • All fixes are mechanical formatting — no logic changes

Fix categories

  • Arrow alignment (WordPress.Arrays.MultipleStatementAlignment)
  • Equals sign alignment (Generic.Formatting.MultipleStatementAlignment)
  • Array indentation (WordPress.Arrays.ArrayIndentation)
  • Yoda conditions (WordPress.PHP.YodaConditions)
  • Short ternary replacement (Universal.Operators.DisallowShortTernary)
  • WP filesystem alternatives (WordPress.WP.AlternativeFunctions)
  • Namespace blank line (PSR2.Namespaces.NamespaceDeclaration)
  • Double quote usage (Squiz.Strings.DoubleQuoteUsage)
  • Post-increment style (Universal.Operators.DisallowStandalonePostIncrementDecrement)
  • PHPCS ignore comments for known false positives

Remaining (19 errors, 6 warnings)

These require manual fixes — they're not auto-fixable by PHPCBF or the custom fixers.

Ran homeboy lint data-machine --fix with parallel PHPCS/PHPCBF.
Fixes include: arrow alignment, equals alignment, array indentation,
Yoda conditions, short ternary replacement, WP filesystem/alternatives,
unused parameter noop insertion, and PHPCS ignore comments.
@github-actions
Copy link

github-actions bot commented Mar 16, 2026

Homeboy Results — data-machine

Lint

⚡ Scope: changed files only

lint (changed files only)

Test

Failure Digest

Test Failure Digest

Autofixability classification

  • Overall: human_needed
  • Autofix enabled: no
  • Autofix attempted this run: no
  • Human-needed failed commands:
    • test
  • Failed commands with available automated fixes:
    • test
  • Automated fixes are disabled for this step. Commands with available fix support in this run: test

Machine-readable artifacts

  • homeboy-lint-summary.json
  • homeboy-test-failures.json
  • homeboy-audit-summary.json
  • homeboy-autofixability.json

⚡ Scope: changed files only

test (changed files only)

Audit

⚡ Scope: changed files only

audit (changed files only)

Tooling versions
  • Homeboy CLI: homeboy 0.78.0+388ce3ee
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: unknown
  • Action: Extra-Chill/homeboy-action@v1

Homeboy Action v1

@chubes4 chubes4 merged commit 7a0e325 into main Mar 16, 2026
2 of 3 checks passed
@chubes4 chubes4 deleted the fix/lint-autofix branch March 16, 2026 22:11
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