Skip to content

Add flower-field, deprecating minesweeper#175

Merged
Anton-4 merged 2 commits into
exercism:mainfrom
BNAndras:add-flower-field
Jul 2, 2025
Merged

Add flower-field, deprecating minesweeper#175
Anton-4 merged 2 commits into
exercism:mainfrom
BNAndras:add-flower-field

Conversation

@BNAndras

@BNAndras BNAndras commented Jul 2, 2025

Copy link
Copy Markdown
Member

This PR adds Flower Field, deprecating Minesweeper. See https://forum.exercism.org/t/suggestion-deprecate-minesweeper-for-flower-field/17967

I'm having some trouble with the template. I'm using the Minesweeper template with some very minor alterations. However roc test fails because Str.replaceEach isn't valid although the Minesweeper tests use Str.replace_each. Is this a version mismatch? Locally, I'm on roc nightly pre-release, built from commit c47a8e9 on Sa 22 Mär 2025 09:02:05 UTC

@Anton-4

Anton-4 commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Thanks for the PR @BNAndras!

However roc test fails because Str.replaceEach isn't valid although the Minesweeper tests use Str.replace_each. Is this a version mismatch?

Yes, this repo is still on Roc alpha2, I'll update it in another PR. The new snake_case way (Str.replace_each) is what you should use.

@Anton-4

Anton-4 commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Yes, this repo is still on Roc alpha2, I'll update it in another PR.

#176

@Anton-4 Anton-4 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @BNAndras!

@Anton-4 Anton-4 merged commit 6ee25ea into exercism:main Jul 2, 2025
2 checks passed
@BNAndras BNAndras deleted the add-flower-field branch July 2, 2025 13:51
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.

2 participants