ucberkeley / bce

Berkeley Common Environment provides a common Linux computational environment for classwork and research.
Apache License 2.0
13 stars 6 forks source link

Make vim have sensible defaults #15

Open davclark opened 9 years ago

davclark commented 9 years ago

Tim Pope has a github repo for this, we can just get this directly: https://raw.githubusercontent.com/tpope/vim-sensible/master/plugin/sensible.vim

ryanlovett commented 9 years ago

I like some of his choices, but setting fileformats+=mac might not be appropriate for Windows folks who will be transferring files created in BCE to their host.

Configuring editor preferences could be a minefield, but I do think it is worth enhancing vim for people new to UNIX/vi.

davclark commented 9 years ago

Note that fileformats+=mac just enables trying the mac format, which, sadly, is still produced by default by some microsoft products on OS X (e.g., CSV files on the Mac). I don't care much either way, but tpope asks that if you aren't willing to adopt sensible.vim, that you file an issue - as all values are supposed to be sensible defaults!

ryanlovett commented 9 years ago

Perhaps he has done a lot of research on it then. The help info on 'fileformats' says, "when set to one name, that format will be used whenever a new buffer is opened." So a new file in BCE will default to fileformats=mac. But if he and his cloners have tested the corner cases then our work is done.

I'll peruse its repo issues and if there are no red flags then I can add it.

Ryan

On Wed, Jan 21, 2015 at 5:34 PM, Dav Clark notifications@github.com wrote:

Note that fileformats+=mac just enables trying the mac format, which, sadly, is still produced by default by some microsoft products on OS X (e.g., CSV files on the Mac). I don't care much either way, but tpope asks that if you aren't willing to adopt sensible.vim, that you file an issue - as all values are supposed to be sensible defaults!

— Reply to this email directly or view it on GitHub https://github.com/ucberkeley/bce/issues/15#issuecomment-70956923.