Skip to content

[Minesweeper - (Soon to be Flower Field)] draft approaches - #3663

Closed
colinleach wants to merge 2 commits into
exercism:mainfrom
colinleach:approach-minesweeper
Closed

[Minesweeper - (Soon to be Flower Field)] draft approaches#3663
colinleach wants to merge 2 commits into
exercism:mainfrom
colinleach:approach-minesweeper

Conversation

@colinleach

Copy link
Copy Markdown
Contributor

Just the introduction.md in this case. I couldn't identify such clearly different approaches that would justify separate sections.

I'll read this through again on Saturday, so there may be some minor corrections added.

@colinleach

Copy link
Copy Markdown
ContributorAuthor

I made a few tweaks but couldn't think of anything major,

Over to you for comments. Meanwhile, I'll see how many more of these I can get drafted before this burst of enthusiasm drains away.

@ABcDexterABcDexter left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These look good :)

@BethanyG

Copy link
Copy Markdown
Member

@ABcDexter -- Reviews in this repo should only be done by the maintainer team. Thanks!

@exercismexercism locked as off-topic and limited conversation to collaborators Oct 26, 2024
@BethanyG

Copy link
Copy Markdown
Member

Given this change, going to take this back to draft. After the deprecation/relaunch, we can then explain and revise this.

@BethanyG
BethanyG marked this pull request as draft June 25, 2025 17:24
@BethanyGBethanyG changed the title [Minesweeper] draft approaches[Minesweeper - Soon to be Flower Field] draft approachesJun 25, 2025
@BethanyGBethanyG changed the title [Minesweeper - Soon to be Flower Field] draft approaches[Minesweeper - (Soon to be Flower Field)] draft approachesJun 25, 2025
@exercismexercism unlocked this conversation Jun 25, 2025
@BethanyG

Copy link
Copy Markdown
Member

@colinleach So I think the thing to do here is to take what we have and copy it into the flower-field exercise. (i.e. a new PR).

We can then change the mine and minesweeper references and re-review. We can then close this PR in favor of the new flower field one.

Happy to have you do that, or I can. Either works. 😄 Just let me know!

@colinleach

Copy link
Copy Markdown
ContributorAuthor

Sounds simple enough that I'll do it later today.

@colinleach

Copy link
Copy Markdown
ContributorAuthor

It's funny to re-read this 15 months later.

Meanwhile, I'll see how many more of these I can get drafted before this burst of enthusiasm drains away

That draining didn't take long, for either of us!

@colinleach

Copy link
Copy Markdown
ContributorAuthor

Replaced by #3935

Sign up for freeto 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

@colinleach@BethanyG@ABcDexter