10up / wpcs-action

GitHub Action to help you lint your PHP without additional dependencies within your codebase
MIT License
54 stars 13 forks source link

Install the VIPCS package from a specific branch #37

Closed dkotter closed 12 months ago

dkotter commented 12 months ago

Description of the Change

We currently install the VIPCS package from the main branch, which means we're always pulling in the latest changes. They recently made some major changes to go along with the 3.0.0 release of the WPCS package and this breaks things with how we currently install these packages.

Similar to the change in #34, this PR fixes things by installing a specific branch of the VIPCS package. This allows things to run properly again, though same as noted in #34, we should look to refactor how we install these packages to use composer instead, to ensure we can use the latest versions of things (so this fix should be looked at as temporary).

How to test the Change

Can see that with this change in place, issues are properly flagged in ClassifAI

Changelog Entry

Fixed - Clone the 2.3.3 tagged release of the VIPCS package to ensure running VIP scans works.

Credits

Props @dkotter, @TylerB24890

Checklist: