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

Add an 'initial' element to the Drupal Scaffold 'extras' section #44

Merged
merged 1 commit into from
Aug 29, 2016

Conversation

greg-1-anderson
Copy link
Collaborator

This allow files to be scaffolded iff the target file does not yet exist (e.g. to copy from a default file to the user-customized file on first install only).

One important example of this is the settings.php file. In drops-8, Pantheon supplies an initial settings.php file; of course, Drupal makes this file read-only during the installation process. We want to be able to scaffold the settings.php file on the initial run, but on subsequent runs it would cause an error if we attempted to copy over this file. Using the 'initial' files list introduced in this PR achieves this goal.

…allow files to be scaffolded iff the target file does not yet exist (e.g. to copy from a default file to the user-customized file on first install only).
@greg-1-anderson
Copy link
Collaborator Author

@webflo: Any objections or requests for this?

@greg-1-anderson
Copy link
Collaborator Author

Merging.

@greg-1-anderson greg-1-anderson merged commit 617fa3a into master Aug 29, 2016
@greg-1-anderson greg-1-anderson deleted the initial-scaffold branch August 29, 2016 16:56
@webflo
Copy link
Member

webflo commented Aug 29, 2016

Hmm, this trigger a new tag on Packagist https://packagist.org/packages/drupal-composer/drupal-scaffold#999.0.1464132628 :/. Pinged @derhasi to fix it

@greg-1-anderson
Copy link
Collaborator Author

A failed build left that tag sitting around in my repository, and I accidentally pushed it with --tags. I deleted the tag from github right away, but I guess Packagist held on to the reference. :(

@heddn heddn mentioned this pull request Jan 23, 2017
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.

None yet

2 participants