Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ignore editor temp files #27

Closed
pgaskin opened this issue May 24, 2020 · 6 comments · Fixed by #30
Closed

Ignore editor temp files #27

pgaskin opened this issue May 24, 2020 · 6 comments · Fixed by #30
Assignees
Milestone

Comments

@pgaskin
Copy link
Owner

pgaskin commented May 24, 2020

MR:post-3992019

@pgaskin pgaskin added this to the v0.1.0 milestone May 24, 2020
@pgaskin
Copy link
Owner Author

pgaskin commented May 24, 2020

  • Nano: *.swp
  • Vim: *~, *.swp, *.swo (it can also create more sequential extensions, but those aren't likely to happen anyways)
  • Gedit: *~
  • Emacs: *~, #*#

Any others?

@pgaskin
Copy link
Owner Author

pgaskin commented May 24, 2020

I should probably also exclude OS-specific files too:

  • macOS: .DS_Store*, .Spotlight-V*, ._*
  • Windows: thumbs.db, Thumbs.db, desktop.ini

@NiLuJe
Copy link
Collaborator

NiLuJe commented May 24, 2020

Kate does .${file}.kate-swp (i.e., besides the extension, it also makes it hidden).

@pgaskin pgaskin self-assigned this May 25, 2020
@pgaskin
Copy link
Owner Author

pgaskin commented May 25, 2020

I think I'll exclude all dotfiles, the Windows stuff, and stuff ending with .swp, .swo, ~ or #.

@shermp
Copy link
Collaborator

shermp commented May 25, 2020

Wouldn't it have been easier to have a defined config format? Either a custom extension, and/or file signature?

Something like config.nm?

@pgaskin
Copy link
Owner Author

pgaskin commented May 25, 2020

That was my original plan, but I realized it would be easier for Windows users if I allowed any extension (most don't know how to rename files to / save files under different extensions, and many don't even know files have extensions).

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

Successfully merging a pull request may close this issue.

3 participants