-
Notifications
You must be signed in to change notification settings - Fork 9
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
Don't auto-scaffold #36
Conversation
I can't seem to point to this PR.
|
"no-auto-scaffold" is an invalid version? |
I think we may still need this for Islandora-Devops/islandora-playbook#177 |
I needed change the owner of |
@whikloj I can update those in the playbook |
It is more of an implementation detail, I think it is worth considering but was not a deal breaker. Just wanted to note it incase anyone mentioned it. |
Eh it's all related. I'd prefer to take care of it now. This permissions thing has been around for long enough. Not being able to composer require out of the box is a pretty 💩 user/dev experience. |
GitHub Issue: Goes with Islandora-Devops/islandora-playbook#147
What does this Pull Request do?
Unregisters the scaffold scripts so they are no longer run every time we update.
How should this be tested?
Set
drupal_composer_project_package: "islandora/drupal-project:dev-no-auto-scaffold"
ingroup_vars/webserver/drupal.yml
and provision a server.Interested parties
@elizoller, @Islandora/8-x-committers