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

Apply PSR Code Styling Standard to the library #89

Merged
merged 1 commit into from
Nov 12, 2018

Conversation

guzzilar
Copy link
Contributor

@guzzilar guzzilar commented Nov 12, 2018

1. Objective

There are bunch of legacy codes that don't comply with the PSR-2 Code Standard.

This pull request is aiming to clean up those legacy codes.

2. Description of changes

Apply PSR-2 Code Standard to the library.

However, there is still 1 leftover place that won't be updated in this pull request, which is OmiseException class.

The reason is that this class will be refactor later on in the coming pull request. (as it requires more than just update code styling but more on a refactoring part. So I leave it for an appropriate pull request).
screen shot 2561-11-12 at 14 33 44

@guzzilar guzzilar force-pushed the apply-psr-standards branch from 8f74603 to 53110c5 Compare November 12, 2018 07:33
@guzzilar guzzilar changed the title [WIP] Apply PSR Code Styling Standard to the library Apply PSR Code Styling Standard to the library Nov 12, 2018
@guzzilar guzzilar merged commit 648dfb2 into master Nov 12, 2018
@guzzilar guzzilar deleted the apply-psr-standards branch November 12, 2018 08:03
@guzzilar guzzilar mentioned this pull request Dec 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants