Skip to content

Conversation

Spea
Copy link
Contributor

@Spea Spea commented Mar 13, 2017

This will fix #31

@coveralls
Copy link

coveralls commented Mar 13, 2017

Coverage Status

Coverage remained the same at 91.514% when pulling bebebbb on rebuy-de:fix-composer into d90fddc on optimizely:master.

@aliabbasrizvi
Copy link
Contributor

Thanks for submitting the change @Spea . In order to proceed it would be required to sign the Contributor License Agreement here.

@Spea
Copy link
Contributor Author

Spea commented Mar 15, 2017

Done

@aliabbasrizvi
Copy link
Contributor

Cool, thanks for doing that. I notice that you are trying to merge into master. We need to merge it in devel first and then for the next release we will merge into master as there are some other changes as well.

@Spea Spea changed the base branch from master to devel March 16, 2017 09:54
@Spea
Copy link
Contributor Author

Spea commented Mar 16, 2017

Rebased onto devel and changed base branch, sorry for the inconvenience

@coveralls
Copy link

coveralls commented Mar 16, 2017

Coverage Status

Coverage remained the same at 91.514% when pulling 59257e4 on rebuy-de:fix-composer into 2e8f1d4 on optimizely:devel.

@aliabbasrizvi
Copy link
Contributor

Thanks for doing that @Spea

Will ping you when we have a release going. It should be in the next couple of days.

@aliabbasrizvi aliabbasrizvi merged commit d818b99 into optimizely:devel Mar 20, 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.

3 participants