search

WordPress / theme-review-action

Other
31 stars 10 forks source link

Reports should not include `test-theme` #64

Closed dd32 closed 2 years ago

dd32 commented 3 years ago

Currently reports generated by the action include the path as themes/test-theme/ which leaves some people confused.

There's two options I see:

Example: https://themes.trac.wordpress.org/ticket/105885#comment:1

StevenDufresne commented 2 years ago

I think the quickest way would be to rewrite the output from the action. Keeping the folder generic in this project makes the code base simpler.

dd32 commented 2 years ago

@StevenDufresne In that case, we can close this as https://github.com/WordPress/wordpress.org/commit/fecfd88195d52c33ce9c5101d2de35bd9abc805c handles that rewrite.