Merge 4dddcc9 into bc6fe3f

README.md

@@ -48,12 +48,12 @@ Usage: shallow-backup [OPTIONS]
 
 Options:
   --add_dot                Add a dotfile or dotfolder to config by path.
-  -all                     Full back up.
   -configs                 Back up app config files.
   -delete_config           Delete config file.
   -destroy_backup          Delete backup directory.
   -dotfiles                Back up dotfiles.
   -fonts                   Back up installed fonts.
+  -full_backup             Full back up.
   --new_path TEXT          Input a new back up directory path.
   -no_splash               Don't display splash screen.
   -old_path                Skip setting new back up directory path prompt.

shallow_backup/__main__.py

@@ -12,7 +12,7 @@
 # custom help options
 @click.command(context_settings=dict(help_option_names=['-h', '-help', '--help']))
 @click.option('--add_dot', default=None, help="Add a dotfile or dotfolder to config by path.")
-@click.option('-all', is_flag=True, default=False, help="Full back up.")
+@click.option('-full_backup', is_flag=True, default=False, help="Full back up.")
 @click.option('-configs', is_flag=True, default=False, help="Back up app config files.")
 @click.option('-delete_config', is_flag=True, default=False, help="Delete config file.")
 @click.option('-destroy_backup', is_flag=True, default=False, help='Delete backup directory.')
@@ -31,7 +31,7 @@
 @click.option('-separate_dotfiles_repo', is_flag=True, default=False, help="Use if you are trying to maintain a separate dotfiles repo and running into issue #229.")
 @click.option('-show', is_flag=True, default=False, help="Display config file.")
 @click.option('--version', '-v', is_flag=True, default=False, help='Display version and author info.')
-def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, new_path,
+def cli(add_dot, full_backup, configs, delete_config, destroy_backup, dotfiles, fonts, new_path,
         no_splash, old_path, packages, reinstall_all, reinstall_configs,
         reinstall_dots, reinstall_fonts, reinstall_packages, remote,
         separate_dotfiles_repo, show, version):
@@ -46,10 +46,10 @@ def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, n
 
 	# Process CLI args
 	admin_action = any([add_dot, delete_config, destroy_backup, show, version])
-	has_cli_arg = any([old_path, all, dotfiles, packages, fonts, configs,
+	has_cli_arg = any([old_path, full_backup, dotfiles, packages, fonts, configs,
 	                   reinstall_dots, reinstall_fonts, reinstall_all,
 	                   reinstall_configs, reinstall_packages])
-	skip_prompt = any([all, dotfiles, configs, packages, fonts, reinstall_packages, reinstall_configs, reinstall_dots,
+	skip_prompt = any([full_backup, dotfiles, configs, packages, fonts, reinstall_packages, reinstall_configs, reinstall_dots,
 	                   reinstall_fonts])
 
 	safe_create_config()
@@ -123,9 +123,9 @@ def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, n
 			reinstall_dots_sb(dotfiles_path)
 		elif reinstall_all:
 			reinstall_all_sb(dotfiles_path, packages_path, fonts_path, configs_path)
-		elif all:
+		elif full_backup:
 			backup_all(dotfiles_path, packages_path, fonts_path, configs_path, skip=True)
-			git_add_all_commit_push(repo, "all")
+			git_add_all_commit_push(repo, "full_backup")
 		elif dotfiles:
 			backup_dotfiles(dotfiles_path, skip=True)
 			git_add_all_commit_push(repo, "dotfiles", separate_dotfiles_repo)
@@ -143,7 +143,7 @@ def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, n
 		selection = main_menu_prompt().lower().strip()
 		selection_words = selection.split()
 		if selection.startswith("back up"):
-			if selection_words[-1] == "all":
+			if selection_words[-1] == "full_backup":
 				backup_all(dotfiles_path, packages_path, fonts_path, configs_path)
 				git_add_all_commit_push(repo, selection_words[-1])
 			elif selection_words[-1] == "dotfiles":
@@ -167,7 +167,7 @@ def cli(add_dot, all, configs, delete_config, destroy_backup, dotfiles, fonts, n
 				reinstall_fonts_sb(fonts_path)
 			elif selection_words[-1] == "dotfiles":
 				reinstall_dots_sb(dotfiles_path)
-			elif selection_words[-1] == "all":
+			elif selection_words[-1] == "full_backup":
 				reinstall_all_sb(dotfiles_path, packages_path, fonts_path, configs_path)
 		elif selection.endswith("config"):
 			if selection == "show config":

shallow_backup/compatibility.py

@@ -3,17 +3,17 @@
 
 
 def get_os_name():
+	"""Returns name of OS"""
 	return platform.system().lower()
 
 
 def get_home():
+	"""Returns path to ~"""
 	return os.path.expanduser('~')
 
 
 def get_config_paths():
-	"""
-	Returns a dict of config paths for the correct OS.
-	"""
+	"""Returns a dict of config paths for the correct OS."""
 	if "darwin" == get_os_name():
 		sublime2_path = os.path.join(get_home(), "Library/Application Support/Sublime Text 2")
 		sublime3_path = os.path.join(get_home(), "Library/Application Support/Sublime Text 3")
@@ -51,6 +51,7 @@ def get_config_paths():
 
 
 def get_fonts_dir():
+	"""Returns default path to fonts on the current platform"""
 	os_name = get_os_name()
 	if os_name == "darwin":
 		return os.path.join(get_home(), "Library/Fonts")
@@ -59,6 +60,7 @@ def get_fonts_dir():
 
 
 def get_applications_dir():
+	"""Returns default path to applications directory"""
 	os_name = get_os_name()
 	if os_name == "darwin":
 		return "/Applications"

shallow_backup/config.py

@@ -24,9 +24,9 @@ def get_config():
 	:return: dictionary for config
 	"""
 	config_path = get_config_path()
-	with open(config_path) as f:
+	with open(config_path) as file:
 		try:
-			config = json.load(f)
+			config = json.load(file)
 		except json.decoder.JSONDecodeError:
 			print_red_bold(f"ERROR: Invalid syntax in {config_path}")
 			sys.exit(1)
@@ -37,17 +37,17 @@ def write_config(config):
 	"""
 	Write to config file
 	"""
-	with open(get_config_path(), 'w') as f:
-		json.dump(config, f, indent=4)
+	with open(get_config_path(), 'w') as file:
+		json.dump(config, file, indent=4)
 
 
 def get_default_config():
 	"""
 	Returns a default, platform specific config.
 	"""
 	return {
-		"backup_path"      : "~/shallow-backup",
-		"dotfiles"         : [
+		"backup_path": "~/shallow-backup",
+		"dotfiles": [
 			".bashrc",
 			".bash_profile",
 			".gitconfig",
@@ -60,7 +60,7 @@ def get_default_config():
 			".zprofile",
 			".zshrc"
 		],
-		"dotfolders"       : [
+		"dotfolders": [
 			".ssh",
 			".vim"
 		],
@@ -74,7 +74,7 @@ def get_default_config():
 			".pypirc",
 			".DS_Store",
 		],
-		"config_mapping"   : get_config_paths()
+		"config_mapping": get_config_paths()
 	}
 
 
@@ -90,18 +90,6 @@ def safe_create_config():
 		backup_config = get_default_config()
 		safe_mkdir(os.path.split(backup_config_path)[0])
 		write_config(backup_config)
-	else:
-		# If it does exist, make sure it's not outdated.
-		# TODO: Move this to upgrade.py
-		with open(backup_config_path) as config:
-			if "[USER]" in config.readline().strip():
-				if prompt_yes_no("An outdated config file has been detected. Would you like to update this?",
-				                 Fore.GREEN):
-					delete_config_file()
-					safe_create_config()
-				else:
-					print_red_bold("ERROR: Outdated config file found.")
-					sys.exit(0)
 
 
 def delete_config_file():

shallow_backup/git_wrapper.py

@@ -14,7 +14,7 @@
 	"fonts": "Back up fonts.",
 	"packages": "Back up packages.",
 	"configs": "Back up configs.",
-	"all": "Full back up.",
+	"full_backup": "Full back up.",
 	"dotfiles": "Back up dotfiles."
 }
 
@@ -54,7 +54,7 @@ def create_gitignore(dir_path, key):
 		if key == "root-gitignore":
 			files_to_ignore = get_config()["default-gitignore"]
 		elif key == "dotfiles-gitignore":
-			pass
+		    files_to_ignore = []
 	with open(os.path.join(dir_path, ".gitignore"), "w+") as f:
 		for ignore in files_to_ignore:
 			f.write("{}\n".format(ignore))