Add a more robust set for `org-capture-templates` to README.org.

bastibe / org-journal

A simple org-mode based journaling mode

BSD 3-Clause "New" or "Revised" License

1.24k stars 123 forks source link

Add a more robust set for `org-capture-templates` to README.org. #344

Closed ghost closed 8 months ago

ghost commented 3 years ago

Replace a straightforward setq with a smarter setf/alist-get.

This should fix #343

bastibe commented 3 years ago

Thank you for your pull request. I see that in addition to the (setf (alist-get, you also changed the defun to cl-labels, and fit the code into use-package.

I would prefer it if you reverted these two changes, or presented them in a separate pull request for discussion. The reason being that cl-labels might not be as familiar to elisp novices as defun, and not everyone uses use-package. The README should focus on being easy to understand first and foremost, I think.