Philippe Crama

https://github.com/jjuliano/practical.org.el/blob/da0da2ad23da2ed91b9a44f94d251467d414c074/practical.org.el#L303-L304 You could consider rewriting this as ```elisp (when (require 'bbdb nil 'noerror) ``` If `bbdb` can't be loaded, this returns `nil`, but if it can (or if `bbdb` is...

https://github.com/juanjosegarciaripoll/emacs-build/blob/b29b161ecb4896257169d18abf4068f55d867d07/emacs-build.sh#L58 would be better written as ``` for f in $inactive; do echo " --without-$f"; done ``` Otherwise the `emacs-build.sh --help` output looks odd (notice the missing dash in the...

https://github.com/juanjosegarciaripoll/emacs-build/blob/b29b161ecb4896257169d18abf4068f55d867d07/emacs-build.sh#L413 If you try to use `--with-cairo`, you will get errors because `--with-cairo` is inserted into the features instead of `cairo`. I propose to replace this line with: ``` --with-*)...

https://github.com/AnthonySuper/jordan/blob/8eafd098dff777cfdcbf0c08490e7aca8eb8bc67/jordan/lib/Jordan/FromJSON/Class.hs#L131 `tyle` → `style`