Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

Fixes #796: Don't exclude local.* files from build artifact. #797

Merged
merged 1 commit into from
Dec 14, 2016
Merged

Fixes #796: Don't exclude local.* files from build artifact. #797

merged 1 commit into from
Dec 14, 2016

Conversation

geerlingguy
Copy link
Contributor

Fixes #796 .

Changes proposed:

  • Don't exclude local.* from default gitignore template for build artifacts.
  • Don't exclude local.* files from default deploy-exclude.txt file.

@grasmash
Copy link
Contributor

We still need to exclude local.settings.php

@geerlingguy
Copy link
Contributor Author

@grasmash ah, good catch. I'll add that as a separate line then, if that's okay.

@geerlingguy
Copy link
Contributor Author

PR has been updated. I tested with one of my codebases, and it seems to exclude the settings file correctly (but not the default/example one).

@grasmash grasmash added the Enhancement A feature or feature request label Dec 14, 2016
@grasmash grasmash merged commit 65fb895 into acquia:8.x Dec 14, 2016
@geerlingguy geerlingguy deleted the fix-gitignore-local branch March 21, 2017 18:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants