frissyn / dstrap.cr Goto Github PK

hi frissyn, its zplusfour
I just found a bug, and I wanna give you some idea :)

Bug:

look here:

this is a directory, and it has a file icon, can you please fix that?

Idea:

the idea is: use the ARGV variable, the ARGV variable is the cli (crystal executable) 's arguments, it has the Array type
like for example create a function that lists the files in a dir:

def list_dir(__dirname)
  # do something...
end

and then use the ARGV variable to list the dir:

# idk how to calculate the array length in Crystal (is it like ruby? .length() ?)
if ARGV.length() == 0 do
  list_dir('.')
else
  list_dir(ARGV[0])
end

# I hope you understand the code :)

this is better than only listing the parent and the current directory.
however, ima try to create a pull request to make these changes, idk bc idk how to calculate the length of an array lol :D

but anyway, good job on this project :)