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

Extend RawMinkContext rather than MinkContext #74

Merged
merged 1 commit into from Sep 18, 2014

Conversation

mlopes
Copy link

@mlopes mlopes commented Sep 18, 2014

No description provided.

@jakzal
Copy link
Contributor

jakzal commented Sep 18, 2014

👍

@debo
Copy link
Contributor

debo commented Sep 18, 2014

No gif, no merge... I think I added that in the contributor documentation :)

@mlopes
Copy link
Author

mlopes commented Sep 18, 2014

Debo, the gif should be put there by the person approving, not the person
making the PR. ;)

Cheers,
Marco Lopes

On 18 September 2014 17:18, Marco De Bortoli notifications@github.com
wrote:

No gif, no merge... I think I added that in the contributor documentation
:)


Reply to this email directly or view it on GitHub
#74 (comment).

@debo
Copy link
Contributor

debo commented Sep 18, 2014

That's never been the policy between me and @alistairstead ;)

@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@mlopes
Copy link
Author

mlopes commented Sep 18, 2014

jzlkgh4

@debo
Copy link
Contributor

debo commented Sep 18, 2014

img_giphy

debo added a commit that referenced this pull request Sep 18, 2014
Extend RawMinkContext rather than MinkContext
@debo debo merged commit 6ff34fe into MageTest:feature/Behat3 Sep 18, 2014
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.

None yet

4 participants