Closed jiegillet closed 3 months 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:
General steps
x:size/{tiny,small,medium,large,massive}
)? (A medium reputation amount is awarded by default, see docs)Any exercise changed
<exercise>/.meta/config.json
(see docs)?<exercise>/.meta/design.md
) need to be updated to document new implementation decisions?Practice exercise changed
prerequisites
, practices
, and difficulty
in config.json
need to be updated?Practice exercise tests changed
<exercise>/.meta/tests.toml
need updating?Automated comment created by PR Commenter 🤖.
@jiegillet rebase/merge main? 1.17 is here 🙂
Excellent suggestion :)
We have a seemingly dead link:
José Valim, the creator of Elixir, explains in his 2012 conference talk how he built the language for applications to be:
But actually the link works. Maybe it's just an anti-bot policy... What do we do in those cases?
.lycheeignore
It's been a while since the last exercise, it's always a pleasure.
I watched the recent 48in24 video about the minesweeper exercise, and I was a bit frustrated not to have seen a higher-order solution using matrix convolution kernels or something, and since this exercise is also about counting neighbors, it was the perfect way to pass my frustration :)