Making WordPress.org

Changeset 8072


Ignore:
Timestamp:
01/11/2019 05:52:54 PM (7 years ago)
Author:
vedjain
Message:

Travis: Remove WordPress-VIP sniffs from phpcs.xml.dist since it looks like they are deprecated.

See https://vip.wordpress.com/documentation/how-to-install-php-code-sniffer-for-wordpress-com-vip/

File:
1 edited

Legend:

Unmodified
Added
Removed
  • sites/trunk/wordcamp.org/phpcs.xml.dist

    r7746 r8072  
    157157    </rule>
    158158
    159     <rule ref="WordPress-VIP">
    160         <!-- We want these for the best practices, but anything specific to VIP's hosting environment is irrelevant. -->
    161         <exclude name="WordPress.VIP.RestrictedFunctions.wp_remote_get_wp_remote_get" />
    162         <exclude name="WordPress.VIP.RestrictedFunctions.file_get_contents_file_get_contents" />
    163         <exclude name="WordPress.VIP.RestrictedFunctions.get_posts_get_posts" />
    164         <exclude name="WordPress.VIP.RestrictedFunctions.switch_to_blog_switch_to_blog" />
    165         <exclude name="WordPress.VIP.RestrictedVariables.cache_constraints___SERVER__REMOTE_ADDR__" />
    166         <exclude name="WordPress.VIP.DirectDatabaseQuery.NoCaching" />
    167         <exclude name="WordPress.VIP.FileSystemWritesDisallow" />
    168         <exclude name="WordPress.VIP.SuperGlobalInputUsage.AccessDetected" />
    169         <exclude name="WordPress.VIP.PostsPerPage.posts_per_page_posts_per_page" />
    170 
    171         <!-- This is too strict. -->
    172         <exclude name="WordPress.VIP.SlowDBQuery.slow_db_query_meta_query" />
    173 
    174         <!-- There are some situations where this is appropriate, so leave it so the developer's discretion. -->
    175         <exclude name="WordPress.VIP.DirectDatabaseQuery.DirectQuery" />
    176 
    177         <!-- High numbers are often used as a sanity limit -->
    178         <exclude name="WordPress.VIP.PostsPerPage.posts_per_page_numberposts" />
    179     </rule>
    180 
    181159    <rule ref="WordPress-Extra">
    182160        <!-- I think it's better to have all the `use` statements come right after the namespace line. -->
Note: See TracChangeset for help on using the changeset viewer.