AuHau / toggl-cli

A simple command-line interface for toggl.com

Home Page:https://toggl.uhlir.dev

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Add `num-entries` option to `ls`

theletterjeff opened this issue · comments

I often use the ls command to see what the exact name was of one of the last N things I worked on. That allows me to type it in exactly when I start a new entry.

It would be great to add a num-entries option to the ls command to constrain the number of rows that are output (without having to specify a datetime range).

That sounds reasonable! Would you be interested in creating a PR for it? The only thing is, I would call the flag --limit as "limit" is a kind of standard term for this, mainly coming from SQL vocabulary.

That sounds reasonable! Would you be interested in creating a PR for it? The only thing is, I would call the flag --limit as "limit" is a kind of standard term for this, mainly coming from SQL vocabulary.

Happy to! I actually have a commit already cooked up. I was just having a bit of difficulty getting the tests to build correctly. Just curious--has the guidance in CONTRIBUTING.md changed at all since it was last updated?

Weeeell, tests are quite broken I believe 🙈 I haven't touched the project for some time, so I don't remember. Just create PR and lets see ;-)