Opened 5 years ago

Closed 4 years ago

Last modified 4 years ago

#4719 closed enhancement (fixed)

Plugin Directory: Check for license on upload

Reported by: ipstenu's profile Ipstenu Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: Priority: normal
Component: Plugin Directory Keywords:

Description (last modified by SergeyBiryukov)

Related to #1944

In order to enforce licenses, the first step is to make sure everyone declares one on upload. This will also catch some people who make a junk readme.

Attachments (1)

1944.diff (5.6 KB) - added by Ipstenu 5 years ago.

Download all attachments as: .zip

Change History (9)

5 years ago

#1 @SergeyBiryukov
5 years ago

  • Description modified (diff)
  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#2 @Otto42
5 years ago

Question: if we're going to start adding this sort of thing, then would it be worth reviving and using that, similar to how we use theme check now?

#3 @Ipstenu
5 years ago

@Otto42 Probably. I'll see if I can port it over now that I have a better idea of how to check for everything via bash.

#4 @Otto42
5 years ago

Well, it's not difficult to do, mainly I was just wondering if there were any other checks we have that we might make modular and move into a plugin form like this, for much the same reason I made theme check initially.

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.

5 years ago

#6 @dd32
4 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 9648:

Plugin Directory: Require a license be specified in the readme.txt.

Props Ipstenu.
Fixes #4719.
See #1944.

#7 @Otto42
4 years ago

In 9655:

Plugin Directory: Reverse the logic on the license check introduced in [9648]. Fixes #4719.

#8 @casiepa
4 years ago

Related: #5126

Note: See TracTickets for help on using tickets.