WordPress.org

Make WordPress Core


Ignore:
Timestamp:
08/17/2018 01:50:26 AM (3 years ago)
Author:
pento
Message:

Coding Standards: Upgrade WPCS to 1.0.0

WPCS 1.0.0 includes a bunch of new auto-fixers, which drops the number of coding standards issues across WordPress significantly. Prior to running the auto-fixers, there were 15,312 issues detected. With this commit, we now drop to 4,769 issues.

This change includes three notable additions:

  • Multiline function calls must now put each parameter on a new line.
  • Auto-formatting files is now part of the grunt precommit script.
  • Auto-fixable coding standards issues will now cause Travis failures.

Fixes #44600.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/query/results.php

    r42343 r43571  
    528528                'child-one',
    529529                'child-two',
    530             ), wp_list_pluck( $posts, 'post_title' )
     530            ),
     531            wp_list_pluck( $posts, 'post_title' )
    531532        );
    532533
     
    544545                'child-three',
    545546                'child-four',
    546             ), wp_list_pluck( $posts, 'post_title' )
     547            ),
     548            wp_list_pluck( $posts, 'post_title' )
    547549        );
    548550
     
    562564                'child-three',
    563565                'child-four',
    564             ), wp_list_pluck( $posts, 'post_title' )
     566            ),
     567            wp_list_pluck( $posts, 'post_title' )
    565568        );
    566569
     
    593596                'child-one',
    594597                'child-two',
    595             ), wp_list_pluck( $posts, 'post_title' )
     598            ),
     599            wp_list_pluck( $posts, 'post_title' )
    596600        );
    597601    }
     
    10641068        $result4 = $this->q->query(
    10651069            array_merge(
    1066                 $args, array(
     1070                $args,
     1071                array(
    10671072                    'has_password'  => true,
    10681073                    'post_password' => '',
     
    10731078        $result5 = $this->q->query(
    10741079            array_merge(
    1075                 $args, array(
     1080                $args,
     1081                array(
    10761082                    'has_password'  => false,
    10771083                    'post_password' => '',
     
    10821088        $result6 = $this->q->query(
    10831089            array_merge(
    1084                 $args, array(
     1090                $args,
     1091                array(
    10851092                    'has_password'  => null,
    10861093                    'post_password' => '',
     
    10921099        $result7 = $this->q->query(
    10931100            array_merge(
    1094                 $args, array(
     1101                $args,
     1102                array(
    10951103                    'has_password'  => true,
    10961104                    'post_password' => 'burrito',
     
    11011109        $result8 = $this->q->query(
    11021110            array_merge(
    1103                 $args, array(
     1111                $args,
     1112                array(
    11041113                    'has_password'  => false,
    11051114                    'post_password' => 'burrito',
     
    11101119        $result9 = $this->q->query(
    11111120            array_merge(
    1112                 $args, array(
     1121                $args,
     1122                array(
    11131123                    'has_password'  => null,
    11141124                    'post_password' => 'burrito',
Note: See TracChangeset for help on using the changeset viewer.