exercism / elixir

Exercism exercises in Elixir.
https://exercism.org/tracks/elixir
MIT License
614 stars 397 forks source link

Refine Take-A-Number #1511

Closed angelikatyborska closed 1 month ago

angelikatyborska commented 1 month ago

Resolves https://github.com/exercism/elixir/issues/1482

This PR reverts https://github.com/exercism/elixir/pull/1486 - see my new comment in that PR to know why.

This PR re-distributes work between step 1 and step 2. Previously, step 1 assumed you would already start the receive loop. That doesn't make sense because there is no unit test for step 1 that could test that you have done this correctly. So I'm moving that to step 2.

github-actions[bot] commented 1 month ago

Thank you for contributing to exercism/elixir 💜 🎉. This is an automated PR comment 🤖 for the maintainers of this repository that helps with the PR review process. You can safely ignore it and wait for a maintainer to review your changes.

Based on the files changed in this PR, it would be good to pay attention to the following details when reviewing the PR:


Automated comment created by PR Commenter 🤖.