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

[+]: fixed performance issue #9

Closed
wants to merge 2 commits into from
Closed

Conversation

voku
Copy link

@voku voku commented Oct 13, 2015

+: fixed code-style | Thx @ PHPStorm ;-)
+: added missing php-docs
+: added typ-hinting comments
+: added .gitattributes (don't fetch everything via composer)

[~]: convert "tabs" (\t) into spaces
[-]: removed unused "use"-statements

PS: And sorry for the big pull-request (bad practice), if you want I can also create a single pull-request for the performance-changes!?

Review on Reviewable

Lars Moelleken added 2 commits October 14, 2015 01:19
[+]: fixed code-style | Thx @ PHPStorm ;-)
[+]: added missing php-docs
[+]: added typ-hinting comments
[+]: added .gitattributes (don't fetch everything via composer)
@alexshelkov
Copy link
Owner

And sorry for the big pull-request (bad practice), if you want I can also create a single pull-request for the performance-changes!?

Yes, pls create another pull request ONLY with performance changes. Try to make as less changes as possible, so it would be easy to merge it.

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