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

Updated .gitignore #3644

Closed
wants to merge 4 commits into from
Closed

Updated .gitignore #3644

wants to merge 4 commits into from

Conversation

sreichel
Copy link
Contributor

@sreichel sreichel commented Nov 11, 2023

Description (*)

  • ignore rector.php config file for now
  • ignore files prefixed with test
  • qodana.yaml from phpstorm
  • .phpunit.result.cache

- ignore rector.php config file (for now)
- ignore files prefixed with test
@github-actions github-actions bot added the ddev label Nov 11, 2023
.gitignore Outdated Show resolved Hide resolved
addison74
addison74 previously approved these changes Nov 12, 2023
@fballiano
Copy link
Contributor

I understand where this comes from but I wouldn't add those extensions to the ignore because on my final projects I may want to have those files in the repo

@sreichel
Copy link
Contributor Author

I'd prefer "ignore all" first and only keep what belongs to core and have a customzed .gitignore in final project.

@sreichel sreichel closed this by deleting the head repository Feb 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants