GH Actions: various tweaks

Related to upstream issue squizlabs/PHP_CodeSniffer 3267 / PR squizlabs/PHP_CodeSniffer 3268

Related to upstream issue squizlabs/PHP_CodeSniffer 3267 / PR squizlabs/PHP_CodeSniffer 3268

Triggering a workflow for a branch manually is not supported by default in GH Actions, but has t...

... on pull requests as it doesn't play nice with required statuses.

The cs2pr tool will allow to display the results from an action run in checkstyle format in-line...

The cs2pr tool will allow to display the results from an action run in checkstyle format in-line...

BCTokens: add `scopeOpeners()` method and update `parenthesisOpeners()`

.. to account for the PHP 8.0 `match` keyword token `T_MATCH`.

These were added to PHPCS as par...

PassedParameters: account for upstream support of named parameters

Upstream PR 3178 to add support for named function call arguments to PHPCS has been merged.

Thi...

PHPCS 4.x | Sync with upstream changes regarding PHP 8 identifier names

Upstream PR 3155 for PHPCS 4.x adds the PHP 8.0 tokens for identifier names to the `Tokens::$fun...

* One test didn't yet take PHP 8 identifier names into account.
* The other did, but contained a...

Upstream PR 3155 for PHPCS 4.x backfills the PHP 8.0 tokenization for identifier names for older...

Upstream PR 3155 for PHPCS 4.x backfills the PHP 8.0 tokenization for identifier names for older...

Tests/UtilityMethodTestCase::skipJSCSSTestsOnPHPCS4: get rid of "risky test" warning

When the `@doesNotPerformAssertions` annotation is used, the test shows as risky when run in com...

PassedParameters::getParameterFromStack(): add extra unit test

... documenting the behaviour when a non-existent parameter is requested.

CI: switch to GitHub Actions

This commit:
* Adds a GH Actions workflow for the CI checks which were previously run on Travis ...

This commit:
* Adds a GH Actions workflow for the CI checks which were previously run on Travis ...

This commit:
* Adds a GH Actions workflow for the CI checks which were previously run on Travis ...

This commit:
* Adds a GH Actions workflow for the CI checks which were previously run on Travis ...

Tests: Xdebug 3 compatibility fix

We are currently limiting the PHPUnit version being used to max PHPUnit 9.2.x due to an issue wi...

PassedParameters: add support for PHP 8.0 named arguments in function calls

While the PHP 8.0 `match` control structure is not yet supported in PHPCS itself, in PHP 8.0, it...

To support named parameters in older PHPCS versions in which named parameters aren't supported b...

PHP 8.0 introduces named function call parameters:
```php
array_fill(start_index: 0, count: 100,...

PassedParameters: add support for anonymous class instantiation parameters

Includes unit tests.

Collections: add new `$arrayOpenTokensBC`, `functionCallTokens()` and `parameterPassingTokens()`

Tests: fix namespace of a test class

Move the token collection which handles which tokens are supported in the PassedParameters class...

Arrays::isShortArray()/Lists::isShortList(): sync with upstream

Upstream PR 3172 has been merged, this updates the previously pulled fix to take this into accou...

Arrays::isShortArray()/Lists::isShortList(): work round PHP8 interpolated string dereferencing

As of PHP 8, text strings containing interpolated variables can be dereferenced, however, the sq...

SpacesFixer: bugfix for new line vs trailing comment

Bug fix for a quite specific situation:
* If a new line was the expected spacing
* AND the "firs...

GetMemberPropertiesTest: use named dataprovider entries

BCFile::getMethodProperties(): sync with PHPCS / 'return_type_end_token' index

The `return_type_end_token` index as a part of the return value for `FunctionDeclarations::getPr...

BCFile::getMethodParameters(): sync with PHPCS / constructor property promotion support

PR 169 added support for PHP 8.0 constructor property promotion to the `FunctionDeclarations::ge...

PHP 8.0 | Union types - sync with PHPCS

PR 168 added support for union types to the `Variables::getMemberProperties()` method and by ext...

Apply the same fix which was previously already applied to the `FunctionDeclarations::getPropert...

Upstream PR 3032 introduces a new `T_TYPE_UNION` token for the `|` in union types.

This token i...

PR 168 added support for union types to the `FunctionDeclarations::getParameters()` method.

Ups...

PR 168 added support for union types to the `FunctionDeclarations::getProperties()` method and b...

Upstream PR 3032 introduces a new `T_TYPE_UNION` token for the `|` in union types.

This token i...

Upstream PR 3032 introduces a new `T_TYPE_UNION` token for the `|` in union types.

This token i...

Operators::isTypeUnion(): account for new T_TYPE_UNION token

Minor adjustment, mostly to the unit tests to allow for the new `T_TYPE_UNION` token as introduc...

Travis: add some extra builds

... to verify the tokenizer safeguards against certain low PHPCS versions.

Arrays::isShortArray()/Lists::isShortList(): add extra unit tests + minor fix

... to safeguard against a PHPCS 2.8.x specific issue, which was already handled correctly in `A...

BCFile::isReference(): sync with PHPCS / arrow function params passed by ref

PR 192 added support for recognizing the reference operator when used for parameters passed by r...

Tests: merge the `isShortArray()` and `isShortList()` tests

A square open/close bracket is either a short array or a short list or neither (= plain square b...

New `Utils\Context` class

This class introduces five new utility methods to examine the context in which an arbitrary toke...

BCFile: sync with PHPCS: support namespace operator in type declarations

PR 180 added support for the namespace keyword used as an operator in type declarations to the `...

PR 180 added support for the namespace keyword used as an operator in type declarations to the `...

PR 180 added support for the namespace keyword used as an operator in type declarations to the `...

Collections/UtilityMethodTestCase: sync with phpcs 3.x/PHP 8.0 identifier name tokens change

Upstream PR 3063 which was merged in PHPCS 3.5.7 effectively "undoes" the PHP 8.0 tokenization f...

Upstream PR 3063 which was merged in PHPCS 3.5.7 effectively "undoes" the PHP 8.0 tokenization f...

Parentheses: improve method documentation

Attempt to make the differences between the methods clearer.

Parentheses: recognize more parentheses owners

PHPCS natively does not consider `isset()`, `unset()`, `empty()`, `exit()`, `die()` and `eval()`...

Various minor code tweaks

BCFile/various methods: compatibility with the PHP 8 identifier name tokenization

UseStatements::splitImportUseStatement(): compatibility with the PHP 8 identifier name tokenization

PassedParameters::hasParameters(): compatibility with the PHP 8 identifier name tokenization

While PHPCS itself won't backfill, nor start supporting the PHP 8 identifier name tokens until P...

As the PHP 8 identifier name tokens will not always exist in all supported PHP/PHPCS versions, t...

The `PassedParameters::hasParameters()` does a token based check on whether or not the passed to...

Namespaces::getType(): compatibility with the PHP 8 identifier name tokenization

FunctionDeclarations::getParameters(): compatibility with the PHP 8 identifier name tokenization

The PHP 8 identifier name tokens can occur as part of a (union) type declaration for a function ...

When the PHP 8 identifier name tokenization is used, the `Namespace::getType()` method basically...

Collections::parameterTypeTokens[BC](): support the PHP 8 identifier name tokens

Collections::propertyTypeTokens[BC](): support the PHP 8 identifier name tokens

Collections::returnTypeTokens[BC](): support the PHP 8 identifier name tokens

Includes adjusted unit test.

This commit implicitly adds support for PHP 8 identifier name toke...

Includes adjusted unit test.

This commit implicitly adds support for PHP 8 identifier name toke...

Includes adjusted unit test.

Collections::namespacedNameTokens(): support the PHP 8 identifier name tokens

Includes adjusted unit test.

This change automatically adds compatibility with the PHP 8.0 iden...