Skip to content
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

Fix CS #8

Merged
merged 1 commit into from Jan 2, 2023
Merged

Fix CS #8

merged 1 commit into from Jan 2, 2023

Conversation

szepeviktor
Copy link
Contributor

PSR-12 suggests this indentation

PSR-12 suggests this indentation
@BenMorel
Copy link
Owner

BenMorel commented Jan 2, 2023

Let's merge this one, but I'll probably wait until brick/coding-standards is set up to use it here for further fixes.

@BenMorel BenMorel merged commit e56eff4 into BenMorel:master Jan 2, 2023
@szepeviktor szepeviktor deleted the patch-1 branch January 2, 2023 21:43
@szepeviktor
Copy link
Contributor Author

All right Ben.
That standard seems very permissive 🙃

@BenMorel
Copy link
Owner

BenMorel commented Jan 2, 2023

😁

Work is in progress in brick/coding-standards#2 but I lacked the time to properly review this so far (and I apologize to @tigitz by the way, I didn't forget you 😞)

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.

None yet

2 participants