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

PhpCSFixer does not work #6

Open
gorkalaucirica opened this issue Oct 8, 2018 · 1 comment
Open

PhpCSFixer does not work #6

gorkalaucirica opened this issue Oct 8, 2018 · 1 comment

Comments

@gorkalaucirica
Copy link
Contributor

Version: 0.7.x

Steps to reproduce in a clean PHP project:

  • composer req lin3s/cs --dev
  • Copy composer.json required scripts and extra sections
  • Use following config (as detailed in docs):
parameters:
    enabled:
        - phpmd
        - phpcsfixer
    name: Issue 
    type: project
    year: 2017
    author: LIN3S
    email: [email protected]
    version: 0.1
    phpmd_path: src
    phpmd_rules:
        - controversial
        - unusedcode
        - codesize
        - naming
    twigcs_path: src
    phpcsfixer_path: src
    phpcsfixer_test_path: tests
    phpcsfixer_file_location: vendor/lin3s/cs/src/LIN3S/CS/
  • Disable null output in LIN3S\CS\Checker\PhpCsFixer
  • Commit src folder

You get the following error:

The "<path-to-project>/vendor/lin3s/cs/src/LIN3S/CS/src" directory does not exist.

@gorkalaucirica
Copy link
Contributor Author

This can be fixed with the following config (note phpcsfixer_file_location):

# This file is auto-generated during the composer install
parameters:
    enabled:
        - phpmd
        - twigcs
        - phpcsfixer
    name: 'Aldi IMA'
    type: project
    year: '2018 - present'
    author: LIN3S
    email: [email protected]
    version: 0.1
    phpmd_path: src
    phpmd_rules:
        - controversial
        - unusedcode
        - codesize
        - naming
    twigcs_path: templates
    phpcsfixer_path: src
    phpcsfixer_test_path: tests
    phpcsfixer_file_location: ''

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant