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.

Sign up for GitHub

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

Jump to bottom

CS fix and check #36

Closed
wants to merge 4 commits into from
Closed

CS fix and check #36

wants to merge 4 commits into from

Conversation

andreybolonin
Copy link
Contributor

@andreybolonin andreybolonin commented Jan 30, 2018 •
edited
Loading

@andreybolonin andreybolonin changed the title cs fix CS fix and check Jan 30, 2018
@kwn
Copy link
Owner

kwn commented Jun 13, 2018

Hi @andreybolonin
I'm not planning to fix coding style in legacy codebase at the moment. I'll remove the support for php 5.5 and hhvm soon. Thank you for your contribution, but I'm not going to merge it at the moment as there are conflicts to solve.

@kwn kwn closed this Jun 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andreybolonin @kwn