Skip to content

minesweeper: Add parameter to exercise placeholder. Closes #599 #694

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

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

peterblazejewicz
Copy link
Contributor

As discussed in #509

Thanks!

Copy link

@jankaifer jankaifer left a comment

Choose a reason for hiding this comment

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

inp is in the example.py but i'm not sure if it's clear what you should expect as an input.
Could you please change it to something longer and easier to understand (that is the goal of #509 ).

@peterblazejewicz
Copy link
Contributor Author

@janetriley Not sure if that is enough, updated/rebased. thanks!

Copy link

@jankaifer jankaifer left a comment

Choose a reason for hiding this comment

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

Now its bit longer than I thought but clear.
Thanks

@ilya-khadykin ilya-khadykin merged commit e0d34a5 into exercism:master Oct 5, 2017
@ilya-khadykin
Copy link
Contributor

Thanks @peterblazejewicz!

@jankaifer thanks for taking a look

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