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

Rename phpstan.neon -> phpstan.dist.neon #5692

Merged

Conversation

jeffwidman
Copy link
Member

Per https://phpstan.org/config-reference#config-file:

The usual practice is to have phpstan.neon.dist or phpstan.dist.neon
under version control, and allow the user to override certain settings
in their environment (on their own computer or on a continuous
integration server) by creating phpstan.neon that’s present in
.gitignore file.

I picked phpstan.dist.neon for consistency with
.php-cs-fixer.dist.php config (#5691).

I didn't add the file to .gitignore because I actually want to know if there starts being a new file under that name and see it in git...

@jeffwidman jeffwidman requested a review from a team as a code owner September 12, 2022 23:08
@jeffwidman
Copy link
Member Author

Confirmed this new config file location is used in https://github.com/dependabot/dependabot-core/actions/runs/3041184101/jobs/4898047993#step:6:6746:

 Note: Using configuration file /opt/composer/v1/phpstan.dist.neon.

@jeffwidman jeffwidman self-assigned this Sep 12, 2022
Per https://phpstan.org/config-reference#config-file:
> The usual practice is to have `phpstan.neon.dist` or `phpstan.dist.neon`
> under version control, and allow the user to override certain settings
> in their environment (on their own computer or on a continuous
> integration server) by creating `phpstan.neon` that’s present in
> `.gitignore` file.

I picked `phpstan.dist.neon` for consistency with
`.php-cs-fixer.dist.php` config (dependabot#5691).

I didn't add the file to `.gitignore` because I actually want to know if
there starts being a new file under that name and see it in git...
@jeffwidman jeffwidman force-pushed the rename-phpstan.neon-phpstan.dist.neon branch from b74b87e to 58e4526 Compare September 13, 2022 17:33
@jeffwidman jeffwidman merged commit 7a187db into dependabot:main Sep 13, 2022
@jeffwidman jeffwidman deleted the rename-phpstan.neon-phpstan.dist.neon branch September 13, 2022 18:03
@pavera pavera mentioned this pull request Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants