Skip to content

Feature/boa unfork#6

Merged
helmo merged 7 commits into7.x-3.xfrom
feature/boa-unfork
Oct 20, 2016
Merged

Feature/boa unfork#6
helmo merged 7 commits into7.x-3.xfrom
feature/boa-unfork

Conversation

@helmo
Copy link
Copy Markdown
Member

@helmo helmo commented Sep 15, 2016

@helmo helmo force-pushed the feature/boa-unfork branch from 76e15e7 to a8d8389 Compare September 15, 2016 18:53
if (!is_null(d($platform)->name) && (d($platform)->name == d()->platform->name)) {
drush_set_option('rename_only', TRUE);
foreach ($new_aliases as $alias) {
drush_log(dt("DEBUG RENAME new_aliases is @this", array('@this' => $alias)));
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's remove these DEBUG logs.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I removed them in 90ba4d6

@jonpugh
Copy link
Copy Markdown
Member

jonpugh commented Oct 14, 2016

Should add tests before we call this one done:

  • Clone Task
  • use client email option, check user1 address.

@helmo helmo merged commit 90ba4d6 into 7.x-3.x Oct 20, 2016
@helmo
Copy link
Copy Markdown
Member Author

helmo commented Oct 20, 2016

@jonpugh can you create a new issue to add these extra tests?

@helmo helmo deleted the feature/boa-unfork branch October 21, 2016 09:06
@helmo
Copy link
Copy Markdown
Member Author

helmo commented Oct 21, 2016

for reference @jonpugh already started a PR for a clone test in aegir-project/tests#11

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