-
Notifications
You must be signed in to change notification settings - Fork 17
Feature : Added support of ls
and 'la` command
#20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
take a look @sapatevaibhav @thedevyashsaini |
hi @thedevyashsaini please take a look around |
hi @sapatevaibhav @thedevyashsaini , please take a look as its pending from three days |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rest looks good though - love the color-coded output and the la split. once the ls fallback + parsing fixes are in, we good to merge imo
Hi @thedevyashsaini , i had pused the changes you had suggested please take a look |
Hi @sapatevaibhav @thedevyashsaini as this PR is already opened for more then 3 weeks ago please look into it |
Hi @thedevyashsaini @sapatevaibhav , please respond |
Hi @Adez017, I am getting some errors when I am on this PR. |
as discussed in #18 / #18 (comment) , had updated the
shelll.rs
file for support of the 'laand
ls` command with different colors and theme .