public/preauth
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: add php-cs-fixer with PSR-12 config and CI check #3

Merged
andrew merged 2 commits from chore/add-php-cs-fixer into main 2026-08-11 09:58:52 -04:00
Conversation 0 Commits 2 Files Changed 61
+2426 -1331
lyra commented 2026-08-11 08:30:46 -04:00
Member
Copy Link
Copy Source

Summary

Adds php-cs-fixer as a dev dependency with PSR-12 configuration, adds a CI check, auto-fixes existing violations, and documents the tooling in the readme.

Changes

  • composer.json — Added friendsofphp/php-cs-fixer to require-dev
  • .php-cs-fixer.dist.php — New config using @PSR12 ruleset, excluding var/ and vendor/
  • .gitea/workflows/tests.yaml — Added Run php-cs-fixer dry-run step between dependency installation and tests
  • readme.md — Added Development section with Code Style and Running Tests subsections
  • .gitignore — Symfony recipe added entries for local php-cs-fixer override and cache
  • 58 PHP source/test files — Auto-fixed PSR-12 violations (formatting only, no logic changes)

Verification

  • ✅ php vendor/bin/php-cs-fixer fix --dry-run — 0 files need fixing
  • ✅ php vendor/bin/phpunit — All 222 tests pass (469 assertions)
## Summary Adds `php-cs-fixer` as a dev dependency with PSR-12 configuration, adds a CI check, auto-fixes existing violations, and documents the tooling in the readme. ## Changes - **`composer.json`** — Added `friendsofphp/php-cs-fixer` to `require-dev` - **`.php-cs-fixer.dist.php`** — New config using `@PSR12` ruleset, excluding `var/` and `vendor/` - **`.gitea/workflows/tests.yaml`** — Added `Run php-cs-fixer` dry-run step between dependency installation and tests - **`readme.md`** — Added Development section with Code Style and Running Tests subsections - **`.gitignore`** — Symfony recipe added entries for local php-cs-fixer override and cache - **58 PHP source/test files** — Auto-fixed PSR-12 violations (formatting only, no logic changes) ## Verification - ✅ `php vendor/bin/php-cs-fixer fix --dry-run` — 0 files need fixing - ✅ `php vendor/bin/phpunit` — All 222 tests pass (469 assertions)
lyra added 2 commits 2026-08-11 08:30:46 -04:00
Fix docs, add .dockerignore, fix base64url padding, fix typo
Sync GitHub / sync (push) Successful in 7s
Details
Tests / test (pull_request) Successful in 42s
Details
6b5a711fa9 
- Add .dockerignore to exclude .git, vendor, var, tests, docs, .env
  and other non-build files from Docker context
- Fix broken base64url padding in src/Data/Payload.php: str_pad was
  a no-op because the length argument was always < string length.
  Replaced with correct str_repeat approach
- Fix typo in bin/franken.sh: digtialadapt → digitaladapt
- Add comment to bin/franken.sh noting it's a dev utility
- Remove config/reference.php from git tracking (auto-generated file)
  and add to .gitignore
- Fix readme.md: env.example → example.env (matches actual filename)
chore: add php-cs-fixer with PSR-12 config and CI check
Sync GitHub / sync (push) Failing after 5s
Details
Tests / test (pull_request) Successful in 49s
Details
cb378e20bc 
- Add friendsofphp/php-cs-fixer to require-dev
- Create .php-cs-fixer.dist.php configured for @PSR12 ruleset
- Add php-cs-fixer dry-run step to CI pipeline
- Auto-fix existing PSR-12 violations
- Document code style tooling in readme.md
andrew force-pushed chore/add-php-cs-fixer from 371d1cebb0 to cb378e20bc 2026-08-11 09:56:57 -04:00 Compare
andrew merged commit c0bda8aeec into main 2026-08-11 09:58:52 -04:00
andrew deleted branch chore/add-php-cs-fixer 2026-08-11 09:58:52 -04:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
andrew (Andrew) lyra
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: public/preauth#3