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

Error: Filenames should be all lowercase with hyphens as word separators. #34

Closed
w3bdesign opened this issue Jun 19, 2020 · 3 comments
Closed

Comments

@w3bdesign
Copy link

Getting the following error:

 Error: Filenames should be all lowercase with hyphens as word separators. Expected phpcs-scan-omtoxh.php, but found phpcs-scan-omTOXH.php (WordPress.Files.FileName.NotHyphenatedLowercase).

My filename is inde-xer.php

Is this a bug or am I misunderstanding something here?

@mrrobot47
Copy link
Member

mrrobot47 commented Jun 20, 2020

That is a known issue, it is happening because vip-go-ci, the core part of this action takes the diff. of PR and puts it in a temp file. That temp file name causes this issue.

This bug will be fixed soon. Till then, please add your phpcs rulsets in phpcs.xml file in the root of your repository. That phpcs.xml will be taken into consideration while running phpcs. You can use this phpcs.xml as a base which ignores the NotHyphenatedLowercase error and add other rulesets you want for your project in it.

@w3bdesign
Copy link
Author

That fixed the issue, thanks.

@dannydover
Copy link

Any ETA on when this will be fixed? I see the workaround, but would prefer to keep that rule in place as it is useful.

Alternatively, is there a way to ignore all filenames that match phpcs-scan-*

Thank you.

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

No branches or pull requests

3 participants