Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Behat] Rename store ships step #4197

Merged

Conversation

lchrusciel
Copy link
Member

Q A
Bug fix? yes
Fixed tickets #4195
License MIT

@lchrusciel lchrusciel force-pushed the rename-store-ships-step branch from 4c90f6d to 0226a46 Compare February 17, 2016 14:16
@lchrusciel lchrusciel changed the title Rename store ships step [Behat] Rename store ships step Feb 17, 2016
@@ -96,7 +96,7 @@ public function storeShipsEverythingForFree(ZoneInterface $zone = null)
}

/**
* @Given /^the store ships everything for free to all available locations$/
* @Given /^the store ships everywhere for free$/
*/
public function theStoreShipsEverythingForFreeToAllAvailableLocations()
Copy link
Contributor

Choose a reason for hiding this comment

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

Method should also be renamed :)

@lchrusciel lchrusciel force-pushed the rename-store-ships-step branch 2 times, most recently from 81d188f to fc5ef25 Compare February 17, 2016 14:51
@lchrusciel lchrusciel force-pushed the rename-store-ships-step branch from fc5ef25 to 11a5c56 Compare February 17, 2016 14:53
@pjedrzejewski pjedrzejewski added the DX Issues and PRs aimed at improving Developer eXperience. label Feb 17, 2016
michalmarcinkowski added a commit that referenced this pull request Feb 17, 2016
@michalmarcinkowski michalmarcinkowski merged commit d76a2b3 into Sylius:master Feb 17, 2016
@michalmarcinkowski
Copy link
Contributor

Thanks Łukasz! 👍

@lchrusciel lchrusciel deleted the rename-store-ships-step branch February 18, 2016 13:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
DX Issues and PRs aimed at improving Developer eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants