Opened 5 weeks ago
Last modified 5 weeks ago
#63436 new enhancement
PHPCS for Class FTP files
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Awaiting Review | Priority: | normal |
Severity: | trivial | Version: | |
Component: | General | Keywords: | has-patch dev-feedback |
Focuses: | coding-standards | Cc: |
Description
Discussion coming from here: #63416
First things first, we need to confirm if it's time to remove PHPCS exclusions for PHPCS before moving forward in PHPCS formatting in the individual tickets (including the latest update [59846] that is actually updating PHPCS in one of the 3 FTP files).
I note this to raise in the next dev-chat
and whatever decision, I will update here either continuing and committing, or dismissing (and maybe reverting [59846]
Change History (3)
This ticket was mentioned in PR #8799 on WordPress/wordpress-develop by @SirLouen.
5 weeks ago
#1
- Keywords has-patch added; needs-patch removed
This ticket was mentioned in Slack in #core by benjamin_zekavica. View the logs.
5 weeks ago
#3
@
5 weeks ago
- Keywords dev-feedback added; 2nd-opinion removed
The two main topics commented in the last <dev-chat>
- @westonruter suggested adding PHPCS checks simply to new lines in PRs (to avoid the late increase of PHPCS number of tickets).
- @joemcgill and @desrosj proposed doing a full audit of current external libs and see which are still maintained upstream before taking further steps in this regard.
We could use this topic for further discussion about this (considering that PemFTP is one of those abandoned libraries that will not receive upstream support anymore).
Since class-ftp
is and will be maintained by WordPress developers for now and the future, it makes sense to open the PHPCS dam because PHPCS, as @desrosj pointed out in the meeting, brings more clarity to the code (and the main reason of the approval of [59846])
But we must assume that this will be on hold until some action taken (like acting on the proposed audit)
Time to open FTP Classes for PHPCS?
Nothing to test here.
Trac ticket: https://core.trac.wordpress.org/ticket/63436