WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#38450 closed defect (bug) (fixed)

Twenty Seventeen: Remove whitelisting comments for PHP_CodeSniffer

Reported by: davidakennedy Owned by:
Priority: normal Milestone: 4.7
Component: Bundled Theme Version: 4.7
Severity: normal Keywords: has-patch
Cc: Focuses:

Description

A few comments to whitelist PHP_CodeSniffer errors are left from the theme's time on GitHub and integration with Travis testing. Those should be removed.

Attachments (1)

38450.1.patch​ (2.4 KB ) - added by davidakennedy 10 years ago.
Remove whitelisting comments for PHP_CodeSniffer

Download all attachments as: .zip

Change History (5)

@davidakennedy
10 years ago

Remove whitelisting comments for PHP_CodeSniffer

#1 @davidakennedy
10 years ago

  • Keywords has-patch added

#2 @karmatosed
10 years ago

  • Resolution → fixed
  • Status new → closed
Version 0, edited 10 years ago by karmatosed (next)

#3 @netweb
10 years ago

  • Milestone Awaiting Review → 4.7

This ticket was mentioned in ​Slack in #themereview by dingo_d. ​View the logs.
9 years ago

Note: See TracTickets for help on using tickets.