Skip to content

Update PHP_CodeSniffer repository link#111

Merged
aik099 merged 1 commit into
aik099:masterfrom
rodrigoprimo:docs/update-phpcs-repo-link
Mar 23, 2026
Merged

Update PHP_CodeSniffer repository link#111
aik099 merged 1 commit into
aik099:masterfrom
rodrigoprimo:docs/update-phpcs-repo-link

Conversation

@rodrigoprimo
Copy link
Copy Markdown
Contributor

PHP_CodeSniffer has moved to a new GitHub organization. The squizlabs/PHP_CodeSniffer repository is now archived and the project is actively maintained at https://github.com/PHPCSStandards/PHP_CodeSniffer.

See: squizlabs/PHP_CodeSniffer#3932

@rodrigoprimo rodrigoprimo force-pushed the docs/update-phpcs-repo-link branch from 75df31f to b1ff751 Compare March 18, 2026 17:59
Copy link
Copy Markdown
Owner

@aik099 aik099 left a comment

Choose a reason for hiding this comment

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

You've changed the owner (from squizlabs into PHPCSStandards( in the link, but both links lead to 404 Page Not Found page.

Please investigate and see how to fix that link while preserving your intention (of link fixing) and original idea behind the link itself.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.73%. Comparing base (7fdbc74) to head (51be1ff).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #111   +/-   ##
=========================================
  Coverage     97.73%   97.73%           
  Complexity      498      498           
=========================================
  Files            29       29           
  Lines          1460     1460           
=========================================
  Hits           1427     1427           
  Misses           33       33           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rodrigoprimo
Copy link
Copy Markdown
Contributor Author

@aik099, I'm sorry for missing that the original and the new link are broken.

While investigating the broken link, I looked into the history of the PHPCS standard to find the right URL to link to. Please correct me if I missed something, but below is what I found.

The PHPCS standard was removed as a standalone installable standard in July 2016 (commit f391069), when its rules were folded into the project-level phpcs.xml.dist file. That means the path CodeSniffer/Standards/PHPCS no longer exists in either repo.

I don't have much of the context of why this repo linked to the PHPCS standard. I opened this PR as part of an effort to update links referencing the old repo to the new one in several repositories.

Given all that, do you prefer that the link points to the PHPCS standard on the squizlabs/2.9 branch (the last version where it existed as a standalone standard as far as I can see) or something else?

Thanks

@aik099
Copy link
Copy Markdown
Owner

aik099 commented Mar 21, 2026

The best match would be https://github.com/PHPCSStandards/PHP_CodeSniffer/tree/2.9/CodeSniffer/Standards/PHPCS link then, because it is the last place where this standard existed in the new repo.

Please use it.

@rodrigoprimo rodrigoprimo force-pushed the docs/update-phpcs-repo-link branch from b1ff751 to 51be1ff Compare March 23, 2026 11:45
@rodrigoprimo
Copy link
Copy Markdown
Contributor Author

Thanks, I applied the change you suggested. Since the changes in this PR are small, I opted to amend the original commit and force push.

@aik099 aik099 merged commit 8061167 into aik099:master Mar 23, 2026
0 of 9 checks passed
@aik099
Copy link
Copy Markdown
Owner

aik099 commented Mar 23, 2026

Merging. Thank you @rodrigoprimo .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants