Making WordPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3080 closed defect (bug) (invalid)

The sample plugin readme file fails the readme validator

Reported by: johnbillion's profile johnbillion Owned by:
Milestone: Priority: normal
Component: Plugin Directory Keywords:
Cc:

Description

At the bottom of https://wordpress.org/plugins/developers/, a link to the "WordPress plugin readme file standard" is provided. This links to a readme file which itself fails the readme validator at https://wordpress.org/plugins/developers/readme-validator/.

Fatal Errors:
No plugin name detected. Plugin names look like: === Plugin Name ===
Warnings:
Requires PHP is missing.
No Contributors listed.

Change History (5)

#1 @Ipstenu
7 years ago

It works if you change it to === MY Plugin Name ===

Hah. Okay, we should change the error to flag that as the default ;)

This ticket was mentioned in Slack in #docs by joyously. View the logs.


7 years ago

#3 follow-up: @Otto42
7 years ago

  • Resolution set to invalid
  • Status changed from new to closed

The plugin name fails because we specifically have the validator looking for the sample "plugin name" header because authors frequently forget to change that and thus call their plugin "Plugin Name".

The Requires PHP thing is new. And there are no Contributors because there is an instruction to put in contributor ID's there instead.

#4 in reply to: ↑ 3 @SergeyBiryukov
7 years ago

  • Keywords needs-patch removed

Requires PHP is missing.

This was not intentional, it should not be a required header. Added a patch on #2952 to remove it from the validator.

#5 @SergeyBiryukov
7 years ago

This was not intentional, it should not be a required header.

On second thought and after some discussions in #2952 and #core-php, the new header should be added to the readme standard. Created #3123 for that.

Note: See TracTickets for help on using tickets.