PracticeCraft / spirit

A series of exercises for Elixir newcomers to practice with language basics and local development.
48 stars 3 forks source link

Drop `Exercises` from the module name #17

Open waseem-medhat opened 1 day ago

waseem-medhat commented 1 day ago

Module names are kinda unwieldy (e.g., one function call looks like this Spirit.Exercises.BinariesStringsAndCharlists.describe_string(s)). So, I suggest we update our structure to mitigate this by replacing the directory name from exercises to just spirit. I've shown it in detail below to make it clear.

Directory structure:

Module name:

An added bonus is that this structure is probably more idiomatic since LSP completion now detects the module name correctly when I use the defmodule snippet.

nikfp commented 1 day ago

I agree that this makes more sense overall. And since having it work with existing tooling is one of our primary goals I think this chance makes sense from an LSP standpoint as well.