exercism / exercism/elm-analyzer

Write analysis for concept exercise `go`

Open
#15 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Elm
Stars
1
Forks
5
PR merge metrics
No merged PRs in 30d

Description

[Source](https://github.com/exercism/elm/tree/main/exercises/concept/go).

I can think of two ways to do checks.
The more general way: make sure that `applyRules` calls `Result.map` and `Result.andThen`

The more specific way: make sure that all of `oneStonePerPointRule captureRule libertyRule koRule` and `changePlayer` are called with `Result.*` instead of in the pattern matching.

The second way is motivated by my own experience of using `changePlayer` in the pattern matching at the end in my first draft. Since the instructions say it should only be done if all passes, it made sense. Of course, using in with `Result.map` is a lot more clever, so maybe other students could be lead to this solution.

Or maybe a mix of both? Check in general and only a special one for `changePlayer`.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.