Context Navigation

← Previous Ticket
Next Ticket →

#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

Attachment 38450.1.patch added

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

https://core.trac.wordpress.org/changeset/38872 fixes.

Last edited 10 years ago by karmatosed (previous) (diff)

#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.

Download in other formats:

Make WordPress Core