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 "work week" focused syntax #1

Open
jorvis opened this issue Feb 19, 2015 · 5 comments
Open

Add "work week" focused syntax #1

jorvis opened this issue Feb 19, 2015 · 5 comments
Assignees

Comments

@jorvis
Copy link
Owner

jorvis commented Feb 19, 2015

@jonathancrabtree suggested the addition of work-week focused list and reporting syntax, like this:

$ qtask list work in this work week
$ qtask list work in last work week

Since these are common reporting intervals

@jonathancrabtree
Copy link

If you're going to go with that syntax then you'll need to clarify what you mean by "work week". Does it include or not include work done on Saturdays and Sundays? And, if it includes it, why not drop the word "work" from both proposed phrases?

@jorvis
Copy link
Owner Author

jorvis commented Feb 19, 2015

The current syntax:

qtask list work in last 1 week

Would already span the weekend if necessary. The proposed 'work week' would be documented to only do Mon - Fri, for those dear people whose work week is actually limited to those days.

I suppose another option to add (and I'm embarrassed I didn't add it right away) is to simply allow the specification of date ranges such as:

qtask list work between 2015-02-10 and 2015-02-17

@jorvis jorvis self-assigned this Feb 19, 2015
@jonathancrabtree
Copy link

Date ranges sound good. With respect to "list work in last 1 week", are the following two things equivalent?

qtask list work in last 1 week
qtask list work in last 7 days

I would expect them to be, in which case I think my question about the work week including the weekends is still valid. That is, I would argue that, in natural language terms, "work in last 1 week" and "work last week" are not equivalent.

@jonathancrabtree
Copy link

OTOH so long as you can specify an explicit date range and so long as the meanings of the various phrases are well-documented I don't think any of this matters too much.

@jorvis jorvis closed this as completed Mar 11, 2015
@jorvis
Copy link
Owner Author

jorvis commented Mar 11, 2015

Initial support for this added. Some ambiguity about whether the range should be inclusive. Feedback welcome.

@jorvis jorvis reopened this Mar 11, 2015
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

No branches or pull requests

2 participants