Skip to content
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

Add num-entries option to ls #313

Closed
theletterjeff opened this issue Oct 29, 2023 · 3 comments · Fixed by #314
Closed

Add num-entries option to ls #313

theletterjeff opened this issue Oct 29, 2023 · 3 comments · Fixed by #314

Comments

@theletterjeff
Copy link
Contributor

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).

@AuHau
Copy link
Owner

AuHau commented Nov 1, 2023

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.

@theletterjeff
Copy link
Contributor Author

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?

@AuHau
Copy link
Owner

AuHau commented Nov 6, 2023

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 ;-)

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 a pull request may close this issue.

2 participants