-
Notifications
You must be signed in to change notification settings - Fork 109
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 blocked predicate #190
Conversation
This can be an alternative to sibling predicates (#105). |
Thanks, this looks like an obviously good idea. :) Sorry for overlooking the PR; feel free to tag me or bump things if I forget. |
59beb87
to
55c83a1
Compare
Resolved the conflict. |
Is there any particular reason this never got merged? Just found this PR looking precisely for a |
I work on these projects in my spare time.
…On Mon, May 16, 2022, 08:13 Marc Fargas ***@***.***> wrote:
Is there any particular reason this never got merged? Just found this PR
looking precisely for a blocked predicate!
—
Reply to this email directly, view it on GitHub
<#190 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAES2FIPEBEJTUG3JEOX6J3VKJCYNANCNFSM4XOWXZ7A>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
I understand. I was mostly asking about feedback on the last comment in this PR to see if I could take the mantle from @akirak and finish the work. Though my Elisp may not yet be good enough! |
I am fine without this feature getting merged. Anyone can define |
4f5fbc4
to
d0acc8c
Compare
55c83a1
to
6afe3be
Compare
@akirak Thanks for your patience. This will be in v0.7. :) |
I have been using this predicate in my config to filter out blocked tasks. I think this can be useful to others, so I am sending a PR.
In most cases, I combine this predicate with negation as follows: