Opened 20 months ago

Last modified 20 months ago

#6108 new defect (bug)

Add automated guideline checks and constant feedback to plugin authors about it

Reported by: dd32's profile dd32 Owned by:
Milestone: Priority: low
Component: Plugin Directory Keywords:

Description (last modified by dd32)

Currently a lot of plugin guidelines are enforced purely through manual processes by reviewers, often in response to complaints/contact from others.

It would be ideal if we had a system similar to Theme Check where automated checks are run over each plugin commit/release.

This could be used to flag serious issues:

  • ERROR: Your plugin name should not include the terms 'WordPress' or 'Plugin'.
  • ERROR: Your plugin name should not need to be 50 words

Feedback on user errors:

  • WARNING: You included 'John Smith' within your 'Contributors:' line in your readme, this should only include usernames of developers, not real names.
  • WARNING: Your plugin slug is 'my-cool-plugin', your textdomain should be the same to be compatible with translations from Detected 'dd-text'.

and less-serious, but annoying things:

  • NOTICE: screenshot-1.png is 4MB in size, consider optimizing it.
  • INFO: Your plugin includes screenshots in the plugin folder, rather than using assets, this results in 5MB extra files within your plugin ZIP.

Due to the different flow for Plugins verse Theme updates, the results of this would need to be handled differently.

My thoughts are a combination of the following:

  • Email to the plugin committers post-version-update
  • Displayed at the top of the Plugins page for plugin committers only
    • and/or Displayed on the Development tab for plugin committers, and everyone else.
  • Logged to a Slack channel, similar to #themereview-firehose and #meta-language-packs

Change History (2)

#1 @dd32
20 months ago

I'm not going to open a new ticket for it right now, but ideally this would also run checks against the Trademark guidelines.

Currently those are only applied upon initial plugin upload, and not on an ongoing basis.

#2 @dd32
20 months ago

  • Description modified (diff)
Note: See TracTickets for help on using tickets.