Skip to content

PHPCS: enforce tab indentation #254

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 6, 2022
Merged

PHPCS: enforce tab indentation #254

merged 1 commit into from
Jun 6, 2022

Conversation

jrfnl
Copy link
Collaborator

@jrfnl jrfnl commented Jun 2, 2022

The PHPCS ruleset as it was, didn't enforce space nor tab indentation.

After discussion with sirbrillig, it became clear that tab indentation is preferred.

This commit adjusts the ruleset to enforce tab indentation and updates all files to use tabs instead of spaces for indentation.

@jrfnl jrfnl requested a review from sirbrillig June 2, 2022 02:22
The PHPCS ruleset as it was, didn't enforce space nor tab indentation.

After discussion with sirbrillig, it became clear that tab indentation is preferred.

This commit adjusts the ruleset to enforce tab indentation and updates all files to use tabs instead of spaces for indentation.
@jrfnl jrfnl force-pushed the feature/cs-enforce-tabs branch from 71586f7 to 2398c78 Compare June 2, 2022 23:29
@jrfnl
Copy link
Collaborator Author

jrfnl commented Jun 2, 2022

Made one minor tweak to the ruleset, no other changes.

@sirbrillig sirbrillig merged commit e048c90 into 2.x Jun 6, 2022
@sirbrillig sirbrillig deleted the feature/cs-enforce-tabs branch June 6, 2022 21: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.

2 participants