Skip to content

Cleanup whitespace#37

Open
Potherca wants to merge 7 commits into
mainfrom
cleanup/whitespace
Open

Cleanup whitespace#37
Potherca wants to merge 7 commits into
mainfrom
cleanup/whitespace

Conversation

@Potherca

@Potherca Potherca commented Jun 24, 2026

Copy link
Copy Markdown
Member

This MR cleans up whitespace to pass PHP CodeSniffer (PHPCS) linting.

Non-code changes only:

  • Indenting
  • Newlines
  • Spaces to tabs

In files where code violations are present, a phpcs:disable comment has been added to avoid the linter from failing. These violations can be resolved when the file(s) in question receive other edits or in a follow-up cleanup MR. (Note: the "disable" is local to the file it is present in)

Additions:

  • An .editorconfig file is included, for editors that support it to automatically use tabs for indenting in PHP files.
  • A PHPCS config has been added, as well as a GitHub action to run PHPCS.
  • A .git-blame-ignore-revs file has been added, to allow git clients that support it to ignore the whitespace commit
    To enable, run git config blame.ignoreRevsFile .git-blame-ignore-revs in the repo once.

Supersedes #30

@Potherca Potherca requested a review from ylebre June 24, 2026 14:52
@Potherca Potherca self-assigned this Jun 24, 2026
@Potherca Potherca mentioned this pull request Jun 24, 2026
@Potherca Potherca marked this pull request as draft June 26, 2026 08:18
@Potherca Potherca force-pushed the cleanup/whitespace branch from c5f6c41 to 66b461d Compare June 26, 2026 08:29
@Potherca Potherca force-pushed the cleanup/whitespace branch from 66b461d to 441aa3a Compare June 26, 2026 09:47
@Potherca Potherca force-pushed the cleanup/whitespace branch from 441aa3a to aaeca41 Compare June 26, 2026 12:28
@Potherca Potherca marked this pull request as ready for review June 26, 2026 12:37
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