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

Added config files that should not be placed into repos by default as th... #488

Merged
merged 1 commit into from
Nov 11, 2013

Conversation

nanadevelopment
Copy link
Contributor

I added two main config files to the gitignore for Expression Engine. I added them for 2 reasons, the constants in config.php change from repo to repo (as they set the URL) and the database.php for obvious security reasons.

… they contain database information and URL's that will vary between local and other servers and can cause conflicts.
@aroben
Copy link
Contributor

aroben commented Nov 11, 2013

Thanks for submitting this! Sorry it's taken so long to take a look.

It does look like lots of ExpressionEngine projects ignore these files, so 👍

aroben added a commit that referenced this pull request Nov 11, 2013
Added config files that should not be placed into repos by default as th...
@aroben aroben merged commit e85ec2d into github:master Nov 11, 2013
drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014
Added config files that should not be placed into repos by default as th...
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
Added config files that should not be placed into repos by default as th...
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 this pull request may close these issues.

2 participants