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

Ensure travis commands run as 'CI' environment. #1715

Merged
merged 2 commits into from
Jun 22, 2017

Conversation

dpagini
Copy link
Contributor

@dpagini dpagini commented Jun 21, 2017

Testing via patch file.

Sorry, meant to test this against my fork. Will reopen if I think it's needed, still testing.

@dpagini dpagini closed this Jun 21, 2017
@@ -6,7 +6,7 @@ set -ev
yaml-cli update:value blt/project.yml project.local.hostname '127.0.0.1:8888'

blt validate:all
blt setup --define drush.alias='${drush.aliases.ci}' --no-interaction
blt setup --define drush.alias='${drush.aliases.ci}' --define enviornment=ci --no-interaction
Copy link
Contributor

Choose a reason for hiding this comment

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

Typo in "enviornment"

@grasmash
Copy link
Contributor

But yes, good idea.

@dpagini
Copy link
Contributor Author

dpagini commented Jun 21, 2017

K, cool... I'll re-open then. Thanks! Should I do the same for the pipelines file at all (not as familiar with that one)?

@dpagini dpagini reopened this Jun 21, 2017
@grasmash
Copy link
Contributor

Yes!

@grasmash grasmash merged commit 0fb0c48 into acquia:8.x Jun 22, 2017
@grasmash
Copy link
Contributor

Added to Pipelines in 4db3020

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.

2 participants