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

#30 Added feature for "Add setting to modify the prefix or suffix of the backup files" #36

Merged
merged 4 commits into from
Oct 2, 2017

Conversation

Abelarm
Copy link
Contributor

@Abelarm Abelarm commented Oct 2, 2017

#30

Added the feature for saving the backup files with a prefix or a suffix.

  • Modified the settings: added a new option "backup_name_mode" where is possible to choose between "suffix" and "predix" or None/false

for this version the prefix/suffix is: "auto-save" it's possible to change modifying the pathhelper.py file.

  • Tested on Windows Sublime Text 3
  • Tested on Mac Sublime Text 3
  • Tested on Linux Sublime Text 3
  • Tested on Windows Sublime Text 2
  • Tested on Mac Sublime Text 2
  • Tested on Linux Sublime Text 2

Future development

  • Adding the possibility to change the suffix/predix string according to the settings

@Abelarm Abelarm changed the title Added feature for "Add setting to modify the prefix or suffix of the backup files" #30 Added feature for "Add setting to modify the prefix or suffix of the backup files" Oct 2, 2017
@akalongman
Copy link
Owner

@Abelarm what if the user does not have a backup_name_mode setting at all?

@Abelarm
Copy link
Contributor Author

Abelarm commented Oct 2, 2017

how do you want for me to handle this?
I could put false as as default

@akalongman
Copy link
Owner

akalongman commented Oct 2, 2017

@Abelarm

I could put false as a default

Yes please

@Abelarm
Copy link
Contributor Author

Abelarm commented Oct 2, 2017

Done 💯

@akalongman akalongman merged commit 63dbaca into akalongman:master Oct 2, 2017
@akalongman
Copy link
Owner

@Abelarm thank you! 👍

@akalongman
Copy link
Owner

Looks like we should update configuration section in the README as well

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

Successfully merging this pull request may close these issues.

2 participants