GH Actions: various tweaks / PHP 8.2 not allowed to fail

PHP 8.2 has been released today :tada: and the `setup-php` action has announced support for PHP ...

... of the bash `date` command in the earlier pulled cache busting.

Modernize/Dirname: magic constants are case-insensitive

Includes a few updated unit tests to safeguard the fix.

GH Actions: add yaml linting and markdown QA

... to loosely safeguard valid and consistent yaml files.

Includes adding a configuration file ...

_Based on a similar workflow previously added to PHPCSUtils._

Remark offers a number of "rules"...

... to be before the adjustments made to the composer.json file.

Changelog: add missing link

Release version 1.0.-RC1

Documentation updates for the release of version 1.0.0-RC1

* Update links to workflows.
* Use re-usable links for links which are used repeatedly.

GH Actions: various tweaks

* Update the PHP version on which the CS run for this package is run to PHP `latest`.
* Remove t...

These steps/directives all relate to testing against PHPCS 4.x, but this package is not being te...

Ref: https://github.com/php-coveralls/php-coveralls/releases

Composer: add PHPCSDevCS to the dependencies

PHPCSDevCS was previously not added to the `require-dev` dependencies as the minimum supported P...

Universal/StaticInFinalClass: handle `static` when used in arrow function

... as long as the arrow function is within an OO construct which can be `final`.

Includes test...

CS/QA: various minor fixes

... now upstream PR 3639 has been merged.

Ref:
* squizlabs/PHP_CodeSniffer 3639

Universal/DuplicateArrayKey: code simplification

No functional changes.

Universal/DuplicateArrayKey: add support for detecting duplicate key PHP cross-version

How array keys for array items without keys are being determined, has changed in PHP 8.0.

Previ...

Universal/DisallowStandalonePostIncrementDecrement: allow for statement ending with close tag

A stand-alone statement doesn't necessarily have to end with a semi-colon. It can also end on a ...

Universal/ForeachUniqueAssignment: improve analysis for foreach list assignments

Universal/DisallowUse[Class|Const|Function]: minor code coverage tweak

Universal/NoReservedKeywordParameterNames: various improvements

This commit:
* Prevents "accidentally correct" handling of code like `foreach ($data as [$id => ...

These tests weren't testing the sniff well enough.

Ignore a statement which should never be reachable (defense in depth code).

... to confirm that properties set via the constructor are also checked.

Properties set via the...

While parameters (variables) are case-sensitive in PHP, keywords are not.

This commit improves ...

... to safeguard bowing out for function declarations without parameters and that the sniff stay...

... which can be made now support for PHPCS < 3.7.1 has been dropped.

The sniff now only listen...

✨ New Modernize.FunctionCalls.Dirname sniff

This new sniff will detect and, when possible, auto-fix two typical code modernizations which ca...

Includes updating CI and other dev related files to take the new standard into account.

Universal/SeparateFunctionsFromOO: update tests for arrow functions

As the sniff uses predefined token collections from PHPCSUtils, it automatically takes PHP 7.4 a...

Universal/SeparateFunctionsFromOO: update tests for enums

Universal/DisallowLonelyIf: bow out early for a specific situation (PHP close tag)

As the sniff uses predefined token collections from PHPCSUtils, it automatically takes PHP 8.1 `...

If an inner "if/else" chain using alternative syntax has a PHP close tag after the `endif` inste...

Universal/DisallowLonelyIf: add extra test

Universal/NoLeadingBackslash: examine imports within a group use statement

.. to cover an edge case which is already handled correctly.

Previously the sniff would only examine the start of a complete import statement, now it will al...

Universal/OneStatementInShortEchoTag: prevent false positive

Universal/RequireFinalClass: various tweaks

Universal/DisallowFinalClass: various tweaks

Universal/RequireAnonClassParentheses: remove redundant condition

Universal/DisallowAnonClassParentheses: remove redundant condition

... and annotate why the condition has been removed, as well as annotating a second likely redun...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

As things were, the sniff would bow out if the opening brace for the class could not be found (y...

In convoluted code, which I hope to never encounter in real life, it _could_ be possible to enco...

When a class is declared as `abstract`, it would attempt to record both for the `abstract`, as w...

Universal/DisallowAlternativeSyntax: bug fix - handle if/elseif statements in one go

As things were, the sniff would examine - and potentially fix - each control structure keyword w...

Universal/DisallowAlternativeSyntax: bug fix - ignore inline HTML in nested closed scopes

... as the HTML in that case is not necessarily echo-ed out when the control structure is execut...

Universal/DisallowAlternativeSyntax: improve error message

Group the setting of the `$code` and `$data` variables for the error message with the conditions...

Improve the readability of the error message by moving the variable part to the end of the message.

When the `$allowWithInlineHTML` property is set to `true`, the error message could be regarded a...

Universal/DisallowAlternativeSyntax: various tweaks

* Minor tweak to the metric name.
* Also record metric for single line/no body control structures.

When a control structure condition is directly followed by a PHP close tag, a semicolon is silen...

While empty "leafs" of control structures is allowed, it is less common, so make the code used f...

GH Actions: bust the cache semi-regularly

Caches used in GH Actions do not get updated, they can only be replaced by a different cache wit...

Universal/Disallow[Long|Short]ListSyntax: improve metrics

Universal/DisallowShortArraySyntax: record metrics

Record both the `yes` as well as the `no`, which makes the metric feature complete.

... as recording just and only the `no` is not a useful metric and adding the logic to also reco...

It's quite straight forward to record informative metrics for this sniff, so let's.

Universal/DisallowShortArraySyntax: don't skip over short lists

Universal/DisallowShortListSyntax: bug fix - don't skip over nested brackets

... as short arrays can contain short lists and those still need to be examined.

Includes test ...