Skip to content

Conversation

Adez017
Copy link
Contributor

@Adez017 Adez017 commented Jul 26, 2025

as discussed in #18 / #18 (comment) , had updated the shelll.rs file for support of the 'laandls` command with different colors and theme .

@Adez017
Copy link
Contributor Author

Adez017 commented Jul 26, 2025

take a look @sapatevaibhav @thedevyashsaini

@Adez017
Copy link
Contributor Author

Adez017 commented Jul 28, 2025

hi @thedevyashsaini please take a look around

@Adez017
Copy link
Contributor Author

Adez017 commented Jul 31, 2025

hi @sapatevaibhav @thedevyashsaini , please take a look as its pending from three days

Copy link
Collaborator

@thedevyashsaini thedevyashsaini left a 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

@Adez017
Copy link
Contributor Author

Adez017 commented Aug 1, 2025

Hi @thedevyashsaini , i had pused the changes you had suggested please take a look

@Adez017 Adez017 requested a review from thedevyashsaini August 9, 2025 14:11
@Adez017
Copy link
Contributor Author

Adez017 commented Aug 9, 2025

Hi @sapatevaibhav @thedevyashsaini as this PR is already opened for more then 3 weeks ago please look into it

@Adez017
Copy link
Contributor Author

Adez017 commented Aug 12, 2025

Hi @thedevyashsaini @sapatevaibhav , please respond

@sapatevaibhav
Copy link
Member

Hi @Adez017, I am getting some errors when I am on this PR.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants