WordPress / plugin-check

A repository for the new Plugin Check plugin from the WordPress Performance and Plugins Team.
https://wordpress.org/plugins/plugin-check/
GNU General Public License v2.0
195 stars 39 forks source link

New Check License for Readme and Plugin Header #454

Closed davidperezgar closed 6 days ago

davidperezgar commented 2 months ago

With this new check solves the issue #436 with this checks:

Fixes #436

davidperezgar commented 1 week ago

I've updated my tests for the new check. Please check @Pacomarchante and @frantorres

davidperezgar commented 1 week ago

Hey, the testing is not working because this reason:ç

[2024-06-22T15:14:50.797Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 429 - {'detail': ErrorDetail(string='Rate limit reached. Please upload with the Codecov repository upload token to resolve issue. Expected time to availability: 1674s.', code='throttled')}

swissspidy commented 1 week ago

This is expected at the moment because PRs from forks don't have access to repository secrets like the codecov token. It can be simply ignored. If you create a PR from a branch directly in this repo, this won't happen.

swissspidy commented 1 week ago

Would be great to add new tests for this change :-)

davidperezgar commented 1 week ago

Yeah, This PR I've made before I have acces to the repo. But it's blocking the merge.

davidperezgar commented 1 week ago

It has the necessary test for different license. How can we solve the merging problem?

swissspidy commented 1 week ago

I just see a lot of code being added but not any new tests. Not a huge deal, but I would have expected more test coverage there.

The failing test does not block anything. Merging is blocked because 2 reviewers requested changes and there are 0 approvals. You can dismiss stale reviews but you still need approvals. Once a PR is approved, it can be merged.

davidperezgar commented 1 week ago

Ok!

frantorres commented 1 week ago

Looks good. Just a detail. The new check was named different_license and there are already other check with the name stable_tag_mismatch, I propose to change it to license_mismatch to follow the convention.

davidperezgar commented 1 week ago

Done!

davidperezgar commented 1 week ago

Changes asked by @mukeshpanchal27 done!