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

bootstrap directory should be ignored or how can we ignore any directory or file? #82

Closed
Jagdish-J-P opened this issue May 18, 2022 · 3 comments · Fixed by #92
Closed

Comments

@Jagdish-J-P
Copy link

Jagdish-J-P commented May 18, 2022

Feature description

When we execute codestyle check/fix - It is also checking bootstrap directory and cache directory in laravel.

I need to know is there any option to ignore specific directory? And if there is no option then how can we ignore specific directory or file?

Envoronment

  • Code Styler Version: 2.1.0
  • PHP Version: 8.1
@andrey-helldar
Copy link
Member

By default, the following directories are ignored:

.git
.github
.idea
node_modules
vendor

At the moment, it is not possible to use custom paths, but we will add this functionality in the near future.

@andrey-helldar andrey-helldar linked a pull request Aug 26, 2022 that will close this issue
@andrey-helldar
Copy link
Member

Sorry for the long wait.

We will not add the ability to use custom paths for exceptions.

  1. php-cs-fixer does not allow you to pass exception commands as parameters - it requires a link to a physical configuration file.
  2. Creating a configuration file on the fly, using a file saved in the project and merging with ours, is a bad idea.

Since we most often work with the Laravel framework, which uses the "bootstrap" directory to store cached data, we decided to add this folder to the exclusions.

This change is released in version 2.6.0.

@Jagdish-J-P
Copy link
Author

Thank you for the update

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

Successfully merging a pull request may close this issue.

2 participants