dthree/cash

`ls -s` is missing

Open

#45 opened on Feb 29, 2016

View on GitHub
 (5 comments) (0 reactions) (0 assignees)JavaScript (7,768 stars) (253 forks)batch import
help wantednon-compliant behavior

Description

I'm very used to typing ls -las. It's in my bones. But the -s option isn't available - so can it please be implemented? Or, since the size is already printed with ls -la, it could also be ignored...

Right now it prints an error message for this switch, even though, technically speaking, it's not an error. It's just not implemented ;)

Contributor guide

Tech stack
javascript
Domain
cli
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Basic JavaScriptUnderstanding of ls command
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40
Research direction
This issue requests the addition of the ` s` flag to the `ls` command in the cash project. The existing `ls` implementation can be found in the repository, likely in a file named `ls.js`. The flag is already used in some form (size is printed with ` la`), so the implementation may involve simply ignoring the flag or adding the size display when ` s` is present. A good starting point is to examine the current command parsing and see how other flags like ` la` are handled. The issue also notes that printing an error for ` s` should be replaced with either a no op or a proper implementation.