pizzatorque

@alphapapa You are completely right about the responsibility of this being of `activities-switch` instead of activities-completing-read. Would the new implementation be more appropriate? I understand changing the default activity offered...

Sorry I have had quite the couple of weeks. I went down a pcase rabbit-hole. What do you think of this implementation? It uses both pcase and cl-remove to simplify...

Would you suggest wrapping that function call into another one like this? This could be used in a couple of other places ```elisp (defun activities--completing-read-actives (prompt) (activities-completing-read :activities (cl-remove-if-not #'activities-activity-active-p...

> I'd prefer not to add another function if it's not necessary. > > What about the suggestion I made here: [#48 (comment)](https://github.com/alphapapa/activities.el/pull/48#issuecomment-2018237587) What problems would using that cause? I...

Oooh this is nice, it's like the pcase I made but it also has error handling and only one call to the activities-completing-read, I think the :test #'eq was what...

Managed to change it :)