technomancy / emacs-starter-kit

[ARCHIVED] this is ancient history
GNU General Public License v3.0
2.86k stars 887 forks source link

put pretty function code into a hook function #85

Closed stesla closed 13 years ago

stesla commented 13 years ago

I don't like the unicode replacements, and it feels cleaner to remove a function from a hook than to remove a font-lock keyword.

There is probably a good deal more duplication removal that could be done here, as the setups are essentially identical.

stesla commented 13 years ago

I finished the extraction of the duplication, and also added a fix for some broken paredit behavior in js-mode.

technomancy commented 13 years ago

Thanks! Could you also submit this as a patch to the v2 branch?

stesla commented 13 years ago

I'd love to submit this to the v2 branch. What is the recommended way of developing/testing against the v2 branch, since it's now designed to be a package?

technomancy commented 13 years ago

You should be able to just use M-x package-install-file to test changes to starter-kit-js provided you have the main starter-kit package installed.