Two identical functions MixHelpers.fetch_repo_contents and MixHelpers.fetch_dir_contents were replaced with a single one MixHelpers.fetch_gh_contents!, with the added ! convention
Renamed fetch_content_object to use the word "download" which makes the distinction between fetching functions (returning the GH API response as-is) vs. this function saving the files to the file system
Reformatted some of the output messages
Removed the TerminalHelpers module, since it's not very beneficial, and simple direct calls to std lib functions in IO.ANSI or Mix.Shell.IO should be good enough
This may need additional refactoring once we start adding exercise fetching from local paths. But for now this should close #5
Summary of changes:
MixHelpers.fetch_repo_contents
andMixHelpers.fetch_dir_contents
were replaced with a single oneMixHelpers.fetch_gh_contents!
, with the added!
conventionfetch_content_object
to use the word "download" which makes the distinction between fetching functions (returning the GH API response as-is) vs. this function saving the files to the file systemTerminalHelpers
module, since it's not very beneficial, and simple direct calls to std lib functions inIO.ANSI
orMix.Shell.IO
should be good enoughThis may need additional refactoring once we start adding exercise fetching from local paths. But for now this should close #5