WordPress.org

Make WordPress Core

Changeset 43641


Ignore:
Timestamp:
09/14/2018 07:54:53 AM (7 months ago)
Author:
flixos90
Message:

Upgrade/Install: Properly exclude the admin/includesSchema.php tests file for the WordPress.DB.PreparedSQL.NotPrepared sniff.

The way this was implemented in [43630] caused the file to be excluded for all sniffs, since PHPCS does not support file-wide ignores of a specific sniff at this point.

Props jrf.
See #44896.

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/phpcs.xml.dist

    r43571 r43641  
    8282        <exclude-pattern>/tests/phpunit/tests/db\.php</exclude-pattern>
    8383    </rule>
     84    <rule ref="WordPress.DB.PreparedSQL.NotPrepared">
     85        <exclude-pattern>/tests/phpunit/tests/admin/includesSchema\.php</exclude-pattern>
     86    </rule>
    8487
    8588    <!-- Whitelist the I18n functions file from issues identified by the I18n sniff
  • trunk/tests/phpunit/tests/admin/includesSchema.php

    r43630 r43641  
    11<?php
    2 // phpcs:ignoreFile WordPress.DB.PreparedSQL.NotPrepared
    32
    43/**
Note: See TracChangeset for help on using the changeset viewer.