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

Switched to PSR-2 #32

Merged
merged 2 commits into from
Sep 1, 2015
Merged

Switched to PSR-2 #32

merged 2 commits into from
Sep 1, 2015

Conversation

lolautruche
Copy link
Contributor

  • added config file for PHP-CS-Fixer

@andrerom
Copy link
Contributor

Seems ezrobot is missing fore this repo (or it has not started yet), add and force push to check?

@lolautruche
Copy link
Contributor Author

It's not configured on this repo

'-blankline_after_open_tag',
'-single_blank_line_before_namespace',
// psr0 has weird issues with our PSR-4 layout, so deactivating it.
'-psr0',

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I usually enable this check on Studio bundles. I didn't have any problems so far.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's because of split with lib/ and mvc. If you don't have that, then no problem

@lolautruche
Copy link
Contributor Author

Rebased and updated.

@andrerom
Copy link
Contributor

+1

lolautruche added a commit that referenced this pull request Sep 1, 2015
@lolautruche lolautruche merged commit 5aff919 into master Sep 1, 2015
@lolautruche lolautruche deleted the psr-2 branch September 1, 2015 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants