Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Bugfix: Ignore backup files #1151

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
5 participants
Contributor

luke-jr commented Dec 1, 2015

Probably there should just be a function somewhere abstracting all these.

Untested; Ruby sucks.

Contributor

harding commented Dec 2, 2015

Hmm. I don't know what text editor you use, but doesn't it have an option to put all of your backup files in the same directory somewhere? For example, in my .vimrc I have:

" Put backup files out of the way
set backupdir=~/.vim/backup//
set directory=~/.vim/swap//
set undodir=~/.vim/undo//

I'm really not fond of modifying code to deal with arbitrary backup files, even if the ~ suffix is pretty standard.

Contributor

luke-jr commented Dec 2, 2015

It is standard to place them in the same directory, and no, I don't see any way to do that with KWrite/Kate.

Contributor

harding commented Jan 5, 2016

I'm tagging this as Won't fix as I admit this can be a problem for some editors, but I think its a problem to be fixed in the editors not in our code. If one of the other site maintainers thinks this is an appropriate solution and wants to merge it, I won't protest.

@harding harding added the Won't Fix label Jan 5, 2016

Contributor

luke-jr commented Jan 6, 2016

A standard best practice among text editors, is somehow a problem to be fixed in the editors? Seriously?

Contributor

paulmadore commented Jan 9, 2016

ACK

With @luke-jr on this. Please merge and add .swo/.swp to the exclusion list. File naming conventions set by editors older than maintainers of a website should not be influenced by said maintainers.

@wbnns wbnns self-assigned this Dec 9, 2016

@wbnns wbnns added the On Hold label Dec 9, 2016

Contributor

wbnns commented Feb 9, 2017

Hello, since this has been idle for over a year without any additional movement, I'm going to go ahead and close this for now. If there is any strong objection or people feel this should be a priority, please let me know and we can revisit. 👍

@wbnns wbnns closed this Feb 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment