WordPress / wporg-theme-directory

15 stars 6 forks source link

Consider open-sourcing the wporg-ratings plugin #7

Closed tellyworth closed 3 months ago

tellyworth commented 1 year ago

If the wporg-ratings plugin was in a public repo, we could install it in this environment and test against it properly instead of using a mock (#1).

Before that could happen, someone needs to do a quick review and see if there are any glaring security or quality issues preventing the code from being made public.

If it's an unacceptable risk then I guess we can just expand the mock class a little as needed for testing.

If there are no blockers, the easiest place to put it for now is probably within this repo. We could move it elsewhere later when the site is launched.

tellyworth commented 1 year ago

After chatting with @dd32, let's go ahead and open-source the plugin. Necessary steps that I can think of off the top of my head:

Anything else needed there? @pkevan why don't you take a shot at that.

pkevan commented 1 year ago

👀 taking a look

pkevan commented 1 year ago

probably going to need commit access to the repo to add the plugin (not needed immediately)

pkevan commented 1 year ago

i've tested out removing the include in https://dotorg.trac.wordpress.org/changeset/19921/ and all seems good so far.