Skip to content

Commit

Permalink
:octocat: switch to PHPCSStandards/PHP_CodeSniffer (squizlabs/PHP_CodeSniffer…
Browse files Browse the repository at this point in the history 
…#3932)
  • Loading branch information
@codemasher
codemasher committed Jan 5, 2024
1 parent 0c47f04 commit c41e89f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions composer.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,9 @@
},
"require-dev": {
"phan/phan": "^5.4",
"phpmd/phpmd": "^2.13",
"phpmd/phpmd": "^2.15",
"phpunit/phpunit": "^10.5",
"squizlabs/php_codesniffer": "^3.8"
"phpcsstandards/php_codesniffer": "^3.8"
},
"autoload": {
"psr-4": {
Expand Down
2 changes: 1 addition & 1 deletion phpcs.xml.dist
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<?xml version="1.0"?>
<ruleset name="php-settings-container-cs"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="vendor/squizlabs/php_codesniffer/phpcs.xsd">
xsi:noNamespaceSchemaLocation="vendor/phpcsstandards/php_codesniffer/phpcs.xsd">
<description>php-settings-container rules for phpcs</description>

<file>examples</file>
Expand Down

2 comments on commit c41e89f

@jrfnl
Copy link

@jrfnl jrfnl commented on c41e89f Jan 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@codemasher Thank you for your support and your enthousiasm embracing the take-over of the PHP_CodeSniffer package.

In contrast to earlier information, arrangements have been made to allow the package to continue under its original name on Packagist. The commit (in the new repo) to rename the package has been reverted.

I'd recommend reverting the Composer reference changes and keeping the changes which refer to the repo URL on GitHub.

Sorry for the confusion and thank you for understanding. I hope you'll enjoy the 3.8.0 release, which was released last month.

@codemasher
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh! Thank you for the heads-up! I just cherry picked this change from another repo of mine and didn't check back for updates in the meantime. I'll change it back in the next iteration. Thank you for your work!

Please sign in to comment.