Brent Westbrook

> error: unexpected argument '--extend-select I,E' found It looks like pre-commit is passing all of `--extend-select I,E` as a single argument to Ruff. I would probably try something like: ```yaml...

Apparently you have to quote the whole argument to avoid that comma splitting behavior, my mistake. YAML is tricky! Glad you got something that works though :) ```pycon >>> import...

Sure, I think it makes sense to add an example like this to the README!

Hi John, thanks for the report! (and for the great alert package that we depend on!) We already expose `org-alert-time-match-string` as a customizable variable; are you suggesting that this would...

Wow thanks for this! I didn't know about org-ql. I need to take a slightly closer look at how it works, but it seems really promising. Is there any reason...

Thanks again for doing this! I've added a few superficial comments, mostly trying to avoid changes unrelated to adding org-ql. I probably won't get to a fuller review until this...

This is still an open pull request that has not been merged into the version of org-alert available on Melpa. You could give it a try by installing the version...

@rickyson96 Sorry for the delay on this. I've come back around on switching to org-ql, and I've just merged a PR adding more tests and some CI. Could you rebase...

Ah tests are failing because org-ql isn't installed. Can you extend `ci.sh` with the new packages that need to be added: https://github.com/spegoraro/org-alert/blob/0bc04cea718387134c37c9fc4c22215adc3f79db/ci.sh#L5-L7 I'm picturing something like this: ``` shell emacs...

Sounds good, no rush at all. On my machine, I just run `make test` in the `test` subdirectory, but I'm not sure how fragile this test setup is. I have...