Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Don't ignore core WordPress files as they are required for the site to run #310

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
Contributor

willmot commented Apr 19, 2012

Only files which can change per install should be excluded.

@willmot willmot Don't ignore core WordPress files as they are required for the site t…
…o run.


Only files which can change per install should be excluded.
3cdfcae
Contributor

willmot commented on Wordpress.gitignore in 3cdfcae Apr 19, 2012

Exclude wp-config.php as it contains per site config settings.

Contributor

willmot commented on Wordpress.gitignore in 3cdfcae Apr 19, 2012

No need for wp-content/cache/supercache/* as it is matched by the rule above.

Contributor

willmot commented on Wordpress.gitignore in 3cdfcae Apr 19, 2012

* isn't needed as whole directory is excluded.

Contributor

willmot commented on Wordpress.gitignore in 3cdfcae Apr 19, 2012

Common location for WordPress backup plugins to store site backups, which obviously shouldn't be in git.

Contributor

willmot commented on Wordpress.gitignore in 3cdfcae Apr 19, 2012

The license and read me shouldn't be ignored, the license for obvious reasons.

Looks like this changes hasn't been applied to Wordpress.gitignore and I don't see this commit in history for some reason...

Contributor

willmot commented Sep 10, 2012

It hasn't been merged in yet.

@willmot willmot closed this Sep 10, 2012

@willmot willmot reopened this Sep 10, 2012

ShepBook commented Oct 5, 2012

Hey @willmot, I attempted to manually merge this in. Could you take a look at 2671086#Wordpress.gitignore and let me know if it doesn't look right?

@ShepBook ShepBook closed this Oct 5, 2012

Contributor

willmot commented Oct 8, 2012

Looks good to me,

Thanks @ShepBook

Contributor

BFTrick commented Oct 15, 2012

I disagree in regards to removing the wp-config.php from the repo. See my comments in my pull request.

@jstasiak jstasiak pushed a commit to jstasiak/gitignore that referenced this pull request Nov 2, 2013

Jared Koumentis (ShepBook) attempting to merge github#310 2671086

@drothmaler drothmaler pushed a commit to drothmaler/gitignore that referenced this pull request May 27, 2014

Jared Koumentis (ShepBook) attempting to merge github#310 e788913
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment