Skip to content

Apply php-cs-fixer #114

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

Merged
merged 2 commits into from
Jan 6, 2017
Merged

Apply php-cs-fixer #114

merged 2 commits into from
Jan 6, 2017

Conversation

bigfoot90
Copy link
Contributor

@bigfoot90 bigfoot90 commented Jan 3, 2017

Sorry, something have gone wrong, and closed my previous PR #112
Please read my comments anyway.

Note
Updated .php_cs file, and re-run php-cs-fixer tool

@ibennetch @nijel Sorry, haven't read your other repository phpmyadmin/phpmyadmin guidelines
Can we discuss about the coding style here?
I'm not enforcing nothing, in the end you are free to accept and merge this PR or not.

@codecov-io
Copy link

codecov-io commented Jan 3, 2017

Current coverage is 99.92% (diff: 100%)

Merging #114 into master will decrease coverage by <.01%

@@             master       #114   diff @@
==========================================
  Files            53         53          
  Lines          3770       3765     -5   
  Methods         184        184          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           3767       3762     -5   
  Misses            3          3          
  Partials          0          0          

Powered by Codecov. Last update 082a1fa...18989fe

* @package SqlParser
* @subpackage Contexts
* @link https://dev.mysql.com/doc/refman/5.0/en/keywords.html
* @see https://dev.mysql.com/doc/refman/5.0/en/keywords.html
Copy link
Contributor

Choose a reason for hiding this comment

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

As this file is autogenerated by tools/ContextGenerator.php, it will be overwritted again unless we fix the generator as well.

@bigfoot90
Copy link
Contributor Author

Updated ContextGenerator.php to match the new CS.
Re-generated contexts files to make sure it works.

@nijel nijel self-assigned this Jan 6, 2017
@nijel nijel merged commit e8595ec into phpmyadmin:master Jan 6, 2017
@nijel
Copy link
Contributor

nijel commented Jan 6, 2017

Merged, thanks for your contribution!

nijel added a commit that referenced this pull request Jan 6, 2017
Issue #114

Signed-off-by: Michal Čihař <michal@cihar.com>
@bigfoot90 bigfoot90 deleted the php-cs-fixer branch January 6, 2017 12:34
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.

3 participants