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
Ads "Ukoliko" as Croatian "Given" alternative #480
Conversation
"Ukoliko" expresses the givens for the test linguistically in more natural way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @banovotz! Can you add a space after the new keyword please?
(As we release these changes, we'll also need to update the gherkin
dialects table here:
https://docs.cucumber.io/gherkin/reference :))
…On Wed., 19 Sep. 2018, 08:09 Aslak Hellesøy, ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Thanks @banovotz <https://github.com/banovotz>! Can you add a space after
the new keyword please?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#480 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AI4pzZmNdNIP6XlUlj-WhTPXNmj2vu2Iks5ucW8MgaJpZM4WtkBL>
.
|
Ads an space behind "Ukoliko " as Croatian Given alternative
Thanks @aslakhellesoy Space added. |
Hi @banovotz, Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾 In return for this generous offer we hope you will:
On behalf of the Cucumber core team, |
Thanks for merging @xtrasimplicity. Whenever you merge a PR, please update the changelog accordingly, following the conventions alteady present. |
Woops, sorry! I'll do that as soon as I can. :)
…On Tue., 25 Sep. 2018, 04:52 Aslak Hellesøy, ***@***.***> wrote:
Thanks for merging @xtrasimplicity <https://github.com/xtrasimplicity>.
Whenever you merge a PR, please update the changelog accordingly,
following the conventions alteady present.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#480 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AI4pzSrxUVOoyeSOKvq3MHiBM58boqMKks5ueSoFgaJpZM4WtkBL>
.
|
Thanks guys for inviting me to Cucumber core team :) |
"Ukoliko" expresses the givens for the test linguistically in more natural way. See: cucumber/common#480
"Ukoliko" expresses the givens for the test linguistically in more natural way. See: cucumber/common#480
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
"Ukoliko" expresses the givens for the test linguistically in more natural way.
Summary
Details
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: