Automattic / Automattic/VIP-Coding-Standards
4.0 | Remove previously deprecated code
- Vorherrschende Sprache
- PHP
- Sterne
- 261
- Forks
- 44
- Ø Merge
- 19 Min.
- Gemergte PRs (30 T.)
- 1
Beschreibung
Removing the deprecated JS sniffs is tracked separately in #442.
List of deprecated code which should be removed in the next major:
- [ ] `WordPressVIPMinimum\Sniffs\Classes\DeclarationCompatibilitySniff::$checkClasses` - deprecated since VIPCS 3.1.0, see #858, related to #234.
- [ ] `WordPressVIPMinimum\Sniffs\Classes\DeclarationCompatibilitySniff::$checkClassesGroups` - deprecated since VIPCS 3.1.0, see #858, related to #234.
- [ ] The complete `WordPressVIPMinimum\Sniffs\Functions\DynamicCalls` sniff - deprecated since VIPCS 3.1.0, see #517 and #865
- [ ] `WordPressVIPMinimum\Sniffs\Performance\TaxonomyMetaInOptionsSniff::$option_functions` - deprecated since VIPCS 3.1.0, see #872, related to #234.
- [ ] `WordPressVIPMinimum\Sniffs\Security\ProperEscapingFunctionSniff::$url_attrs` - deprecated since VIPCS 2.3.1
- [ ] `WordPressVIPMinimum\Sniffs\Security\ProperEscapingFunctionSniff::$attr_endings` - deprecated since VIPCS 2.3.1
- [ ] `WordPressVIPMinimum\Sniffs\Security\ProperEscapingFunctionSniff::attr_expects_url)_` - deprecated since VIPCS 2.3.1
- [ ] `WordPressVIPMinimum\Sniffs\Security\ProperEscapingFunctionSniff::is_html_attr()` - deprecated since VIPCS 2.3.1
Beitragsleitfaden
Rechercherichtung
Beginnen Sie mit der Prüfung der aufgeführten WordPressVIPMinimum-sniffs und ihrer veralteten Mitglieder. Sehen Sie sich anschließend die referenzierten Issues #858, #234, #517, #865 und #872 an, um den Umfang und die Abhängigkeiten zu prüfen. Die JavaScript-sniffs werden separat in #442 verfolgt. Erledigt bedeutet, dass der aufgeführte veraltete PHP-Code entfernt wird, ohne diese separate Arbeit einzubeziehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100