search

doublep / logview

Emacs mode for viewing log files
GNU General Public License v3.0
153 stars 18 forks source link

Should not add itself to auto-mode-alist #29

Closed jscheid closed 5 years ago

jscheid commented 5 years ago

It's not customary for modes to modify auto-mode-alist. Also, auto-mode-alist is for major modes, and yours is a minor mode.

doublep commented 5 years ago

It's not customary for modes to modify auto-mode-alist.

Maybe ten years ago. All of the few modes I checked now do this. Also as a user I want something I install immediately become visible. If it doesn't, it feels broken to me.

Also, auto-mode-alist is for major modes, and yours is a minor mode.

Oh, really.