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

PHPCS/Composer: update PHPCompatibility #18

Merged
merged 1 commit into from
Mar 6, 2020
Merged

PHPCS/Composer: update PHPCompatibility #18

merged 1 commit into from
Mar 6, 2020

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Sep 2, 2019

Composer:

  • wimg/php-compatibility has been abandoned for nearly a year. Use phpcompatibility/php-compatibility instead.
  • Use the latest version of PHPCompatibility.
    You were missing out on a lot of new checks, including the checks to make sure your code is compatible with the upcoming PHP 7.4.
  • Use the latest version of the DealerDirect Composer PHPCS plugin.
    Composer treats minors < 1.0 as majors, so you need to explicitely update.

PHPCS PHPCompatibility ruleset:

  • No need to set the date.timezone. That sniff has been removed for quite a while now.

Refs:

Composer:
* `wimg/php-compatibility` has been abandoned for nearly a year. Use `phpcompatibility/php-compatibility` instead.
* Use the latest version of PHPCompatibility.
    You were missing out on a lot of new checks, including the checks to make sure your code is compatible with the upcoming PHP 7.4.
* Use the latest version of the DealerDirect Composer PHPCS plugin.
    Composer treats minors < 1.0 as majors, so you need to explicitely update.

PHPCS PHPCompatibility ruleset:
* No need to set the `date.timezone`. That sniff has been removed for quite a while now.

Refs:
* https://github.com/PHPCompatibility/PHPCompatibility/releases/
* https://github.com/Dealerdirect/phpcodesniffer-composer-installer/releases
@jrfnl
Copy link
Contributor Author

jrfnl commented Dec 5, 2019

Any chance that this PR can get some love ?

@bastianschwarz
Copy link
Member

Actually yes since I might do some work on the repo soon. You know how it is, private stuff is always pushed back in favor of the "office needs".

@bastianschwarz bastianschwarz merged commit 0b797c1 into codenamephp:master Mar 6, 2020
@jrfnl jrfnl deleted the feature/update-phpcompatibility branch March 7, 2020 14:19
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