lsd icon indicating copy to clipboard operation
lsd copied to clipboard

Published 20 hours ago • verified publisher icon lsd-rs

Allow globs to apply to '..' and '.'

Open Jackenmen opened this issue 2 years ago • 2 comments

Fixes #847

TODO

  • [x] Use cargo fmt
  • [ ] Add necessary tests
  • [x] Add changelog entry
  • [x] Update default config/theme in README (if applicable)
  • [x] Update man page at lsd/doc/lsd.md (if applicable)

Jackenmen avatar May 26 '23 18:05 Jackenmen

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Jackenmen

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

muniu-bot[bot] avatar May 26 '23 18:05 muniu-bot[bot]

I manually tested this and it acts about as I would expect it to - . on the ignore-globs only hides ., .. only hides .., and having both on the list hides both.

Let me know if I should add any unit tests for this. I looked at the existing tests in the file I edited and there didn't seem to be any testing anything to do with ignore_globs so I figured it might not be necessary.

Jackenmen avatar May 26 '23 18:05 Jackenmen