PHPCSStandards / PHP_CodeSniffer

PHP_CodeSniffer tokenizes PHP files and detects violations of a defined set of coding standards.
BSD 3-Clause "New" or "Revised" License
856 stars 53 forks source link

Increase test coverage for `--sniffs` and `--exclude` options #474

Closed fredden closed 3 months ago

fredden commented 4 months ago

Description

While working on https://github.com/PHPCSStandards/PHP_CodeSniffer/pull/344, it was noted that there are no existing tests for the code being changed. This pull request aims to add some tests to cover the existing behaviour.

Suggested changelog entry

Related issues/external references

Related to #344

Types of changes

PR checklist

fredden commented 3 months ago

@jrfnl, I think this is ready for another review now. When we're happier with the final delta, I can rewrite the history so there are fewer commits before this is merged in.

fredden commented 3 months ago

:tada: Yes. A squash-merge via GitHub is fine with me. Thanks for your coaching here too @jrfnl.