Skip to content
This repository was archived by the owner on Jul 1, 2023. It is now read-only.

Restore php 5.3 #159

Closed
wants to merge 61 commits into from
Closed

Restore php 5.3 #159

wants to merge 61 commits into from

Conversation

glensc
Copy link
Contributor

@glensc glensc commented Mar 13, 2021

The PHP 5.3 was removed, because Travis misconfiguration (32855d5). However, seems most PHP 5.3 code is still there. This will attempt to restore such compatibility.

Example:

As the version still won't be tested in CI, such use will be at their own risk, as this project doesn't CI test those versions.

PS: once this project CI is migrated to GitHub actions and setup-php, then PHP 5.3 support cab be enabled in Ci.

grogy and others added 30 commits January 19, 2020 13:35
PHP 7.4 deprecates the use of curly braces for array/string character access.

See: https://wiki.php.net/rfc/deprecate_curly_braces_array_access
This will allow the Nette Tester dependency to download different versions depending on the PHP version the tests are run on.
Fixing issue with autoloader when executing from outside the source directory:
php-parallel-lint#1
This is the same PR as previously pulled in PR JakubOnderka/PHP-Parallel-Lint 144

This makes the release zips used by Packagist smaller as it doesn't ship the test or config files which end-users shouldn't need anyway.

Fixes 11
Adding a build against `nightly` (PHP 8) which, for now, is allowed to fail.

At this time, there is one unit test which fails `ParallelLint.lint.phpt method=testDeprecated`. This will need to be investigated over time. No urgency yet as it may be related to the Nette testing framework dependency and also PHP 8 is still in flux.
grogy and others added 27 commits April 4, 2020 14:08
The current version is `3.5.4` released end of January 2020 and - aside from lots of bug fixes -, it includes compatibility with PHP 7.4.

See: https://github.com/squizlabs/php_codesniffer/releases
The results of PHPCS run will be no different across PHP versions as all sniffs are unit tested against all supported PHP versions, so it is unnecessary to run it against every PHP version, just run it once per build.

I've chosen to run it against PHP 7.4 as that's the fastest PHP version.
* Rename the ruleset to one which PHPCS will recognize automatically.
    This means you no longer need to pass the `--standard` on the command-line.
* Use the `dist` extension to allow for devs to overload the ruleset file locally and gitignore the typical overload file names which are recognized by PHPCS by default.
* Ruleset/Travis: move command-line arguments to the project ruleset.
    ... and scan all PHP files, not just the files in `src`.
* Ruleset: also scan the `phpt` test files.
* Ruleset: enable parallel scanning.
* Ruleset: set the basepath for a cleaner output report.
* Ruleset: add the PHPCS XML scheme.
* Ruleset: selectively exclude the `skip-linting.php` file for the `Generic.PHP.NoSilencedErrors` rule.
Implementation of feature request 115.

The user can now pass longer, multi-part extensions on the command-line and they will be respected.
Example:
`-e php,php.dist`

Fixes 115
This should fix issue 33, by ignoring PHP startup errors completely for any PHP processes started from within the application.
PHP 8.0 has been branched off two months ago, so `nightly` is now PHP 8.1 and in the mean time PHP 8.0 was released last week.

As of today, there is a PHP 8.0 image available on Travis.

This PR adds a new build against PHP 8.0 to the matrix and, as PHP 8.0 has been released, that build is not allowed to fail.
This test is expected to throw a deprecation warning for the PHP4-style class constructor (method named the same as the class).

In PHP 8, the deprecation warning is no longer thrown and PHP4-style constructor method are now treated as _normal_ methods.

This fixes the test to have the correct expectations.
@glensc
Copy link
Contributor Author

glensc commented Mar 13, 2021

Wrong fork, corrected: php-parallel-lint/PHP-Parallel-Lint#51

@glensc glensc closed this Mar 13, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants