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

Fixes #866: Remove default git.remotes url from project.yml on project create. #877

Merged
merged 1 commit into from
Dec 28, 2016

Conversation

shahinam
Copy link
Contributor

Fixes #866

I wanted to set the value as any empty array, like:

git:
    default_branch: master
    remotes: { }

but drupal yaml:update:value can only work with basic values. So I have set it to empty value, like:

git:
    default_branch: master
    remotes:
        - ''

@@ -19,7 +19,6 @@ Here are tasks that are typically performed at this stage:

* Create and deploy an artifact. See [Deployment workflow](deploy.md).

# Ensure git.remotes are set in blt/project.yml. You should replace the default Bolt Acquia Cloud repository with your Acquia Cloud repository and you should add your Github repository
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still need to let people know that they should add their git repository URL here.

@grasmash grasmash added the Enhancement A feature or feature request label Dec 28, 2016
@grasmash grasmash merged commit 1deddbc into acquia:8.x Dec 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants