Automattic / Automattic/VIP-Coding-Standards

Evaluate WPCS VIP issues

Open
#390 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
261
Forks
44
Avg merge
19m
Merged PRs (30d)
1

Description

As part of [WPCS's reorganisation of labels](https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/1454), the `Component: VIP` is being removed, since WPCS `master` no longer contains the VIP category of sniffs.

The following Issues were still open at the time the label was removed, so these need checking to see if they still apply to any code that we now have in VIPCS:

- [x] ~https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/1379~
- [x] ~https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/662~
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/661 (Applicable to WPCS)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/461 (Applicable to WPCS)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/462 (Possibly applicable to WPCS)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/74 (Possibly applicable to WPCS)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/91 (Valid for VIPCS, but not easily sniffable)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/80 (Valid for VIPCS, but not easily sniffable)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/295 (Likely valid for VIPCS)
- [ ] https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/475 (Likely valid for VIPCS)

Contributor guide

Open the contributing guide

Research direction

Start by reading the linked WPCS reorganisation issue 1454, then review the ten open issues listed here, especially 661, 461, 462, 74, 91, 80, 295 and 475. Compare each issue with the current VIPCS and WPCS scope; done means recording whether it still applies and identifying any follow-up needed.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
tooling
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.