Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ECS] Add `only` feature support, as oppose to `skip` #1537

Merged
merged 1 commit into from May 8, 2019

Conversation

Projects
None yet
2 participants
@TomasVotruba
Copy link
Member

commented May 8, 2019

Closes #1526

Include Checkers Only for Some Paths

This feature is the exact opposite of skip. Do you want to run your rule only on new code? Limit it with only parameter:

services:
    Symplify\CodingStandard\Sniffs\Naming\AbstractClassNameSniff: ~

parameters:
    only:
        Symplify\CodingStandard\Sniffs\Naming\AbstractClassNameSniff:
            - 'src/NewCode/*'
@TomasVotruba

This comment has been minimized.

Copy link
Member Author

commented May 8, 2019

@ruudk What about this one? :)

@TomasVotruba TomasVotruba force-pushed the cs-only branch 2 times, most recently from cbb6a8a to 56d4001 May 8, 2019

@TomasVotruba TomasVotruba force-pushed the cs-only branch from 56d4001 to 4202031 May 8, 2019

@TomasVotruba TomasVotruba merged commit 4458203 into master May 8, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@TomasVotruba TomasVotruba deleted the cs-only branch May 8, 2019

@ruudk

This comment has been minimized.

Copy link

commented May 9, 2019

Awesome! Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.