Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 3 years ago

#53004 closed feature request (wontfix)

Coding Standards: Single-Line Code Comment Format Used In Place Of Multi-Line Format.

Reported by: rkradadiya's profile rkradadiya Owned by:
Milestone: Priority: normal
Severity: normal Version: 5.7
Component: Upgrade/Install Keywords: has-screenshots
Focuses: docs, coding-standards Cc:

Description

Hello,

WordPress has a comment structure.
But developers have used a Single-line comment structure in a Multi-line comment in the below file.
wp-admin/update-core.php

Please use multi line comment instead of a Single.
Thanks,
Rajesh

Attachments (1)

Screenshot_1.png (7.9 KB) - added by rkradadiya 4 years ago.

Download all attachments as: .zip

Change History (3)

#1 @SergeyBiryukov
4 years ago

  • Component changed from Users to Upgrade/Install
  • Keywords dev-feedback removed
  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Hi there, thanks for the ticket!

As previously noted in comment:9:ticket:52025, there are many other instances in core of the single-line comment style being used for multi-line comments.

There is a section in the documentation standards that clarifies which format is generally recommended, but that is just a recommendation and not a strict rule.

This was previously corrected for most of inline comments in core in [47122]. I think I opted to keep comments with one or two lines in single line format, and switch comments with three or more lines to the multi-line format.

At this time, until we have a WPCS rule that forbids using this comment style for multi-line comments, I think it makes sense to keep these two-line comments as is.

#2 @SergeyBiryukov
3 years ago

#54757 was marked as a duplicate.

Note: See TracTickets for help on using tickets.