Autoformat all Python in repo with autopep8

CONTRIBUTING.md

@@ -4,6 +4,7 @@
 
 1. Clone repo and make a new branch: `$ git checkout https://github.com/alichtman/shallow-backup -b [name_of_new_branch]`.
 2. Make changes/add new script and test
-3. Update README
-4. Update CONTRIBUTORS.md
-5. Pull Request with comprehensive description of changes
+3. Format all Python code with [autopep8](https://github.com/hhatto/autopep8).
+4. Update README
+5. Update CONTRIBUTORS.md
+6. Pull Request with comprehensive description of changes

CONTRIBUTORS.md

@@ -0,0 +1,3 @@
+# Contributors
+
+[Caleb Jasik] (https://github.com/jasikpark)

README.md

@@ -171,4 +171,5 @@ Once I'd built that functionality, I wanted to have a single backup utility for
 
 1. Clone repo and create a new branch: `$ git checkout https://github.com/alichtman/shallow-backup -b name_for_new_branch`.
 2. Make changes and test
-3. Open Pull Request with comprehensive description of changes
+3. Format all Python code with [autopep8](https://github.com/hhatto/autopep8).
+4. Open Pull Request with comprehensive description of changes

build/lib/constants.py

@@ -1,8 +1,8 @@
 class Constants:
-	PROJECT_NAME='shallow-backup'
-	VERSION='1.3'
-	AUTHOR_GITHUB='alichtman'
-	AUTHOR_FULL_NAME='Aaron Lichtman'
-	DESCRIPTION="Easily create lightweight documentation of installed packages, dotfiles, and more."
-	URL='https://github.com/alichtman/shallow-backup',
-	AUTHOR_EMAIL='aaronlichtman@gmail.com',
+	PROJECT_NAME = 'shallow-backup'
+	VERSION = '1.3'
+	AUTHOR_GITHUB = 'alichtman'
+	AUTHOR_FULL_NAME = 'Aaron Lichtman'
+	DESCRIPTION = "Easily create lightweight documentation of installed packages, dotfiles, and more."
+	URL = 'https://github.com/alichtman/shallow-backup',
+	AUTHOR_EMAIL = 'aaronlichtman@gmail.com',

build/lib/shallow_backup.py

@@ -20,9 +20,9 @@
 
 def print_version_info():
 	version = "{} v{} by {} -> (Github: {})".format(Constants.PROJECT_NAME,
-			                                      Constants.VERSION,
-			                                      Constants.AUTHOR_FULL_NAME,
-			                                      Constants.AUTHOR_GITHUB)
+                                                 Constants.VERSION,
+                                                 Constants.AUTHOR_FULL_NAME,
+                                                 Constants.AUTHOR_GITHUB)
 	line = "-" * (len(version))
 	print(Fore.RED + Style.BRIGHT + line)
 	print(version)
@@ -86,7 +86,8 @@ def make_dir_warn_overwrite(path):
 	Make destination dir if path doesn't exist, confirm before overwriting if it does.
 	"""
 	if os.path.exists(path) and path.split("/")[-1] in ["dotfiles", "packages", "fonts"]:
-		print(Fore.RED + Style.BRIGHT + "Directory {} already exists".format(path) + "\n" + Style.RESET_ALL)
+		print(Fore.RED + Style.BRIGHT +
+		      "Directory {} already exists".format(path) + "\n" + Style.RESET_ALL)
 		if prompt_yes_no("Erase directory and make new back up?", Fore.RED):
 			shutil.rmtree(path)
 			os.makedirs(path)
@@ -95,7 +96,8 @@ def make_dir_warn_overwrite(path):
 			sys.exit()
 	elif not os.path.exists(path):
 		os.makedirs(path)
-		print(Fore.RED + Style.BRIGHT + "CREATED DIR: " + Style.NORMAL + path + Style.RESET_ALL)
+		print(Fore.RED + Style.BRIGHT + "CREATED DIR: " +
+		      Style.NORMAL + path + Style.RESET_ALL)
 	return
 
 
@@ -127,9 +129,11 @@ def copy_dotfolder(dotfolder, backup_path):
 			command = "cp -aRp " + dotfolder + " " + backup_path + "/macOS_Preferences"
 			print(Fore.BLUE + command)
 		elif "Application\ Support" not in dotfolder or "XCode" in dotfolder:
-			command = "cp -aRp " + dotfolder + " " + backup_path + "/" + dotfolder.split("/")[-2]
+			command = "cp -aRp " + dotfolder + " " + \
+				backup_path + "/" + dotfolder.split("/")[-2]
 		elif "Sublime" in dotfolder:
-			command = "cp -aRp " + dotfolder + " " + backup_path + "/" + dotfolder.split("/")[-3]
+			command = "cp -aRp " + dotfolder + " " + \
+				backup_path + "/" + dotfolder.split("/")[-3]
 		sp.run(command, shell=True, stdout=sp.PIPE)
 
 
@@ -156,12 +160,15 @@ def backup_dotfiles(backup_path):
 
 	# get dotfolders and dotfiles
 
-	dotfiles_for_backup = [".bashrc", ".bash_profile", ".gitconfig", ".pypirc", ".shallow-backup", ".zshrc"]
+	dotfiles_for_backup = [".bashrc", ".bash_profile",
+                        ".gitconfig", ".pypirc", ".shallow-backup", ".zshrc"]
 	dotfolders_for_backup = [".ssh/", ".vim/"]
 
 	# Add dotfile/folder for backup if it exists on the machine
-	dotfiles = [file for file in dotfiles_for_backup if os.path.isfile(os.path.join(home_path, file))]
-	dotfolders = [folder for folder in dotfolders_for_backup if os.path.exists(os.path.join(home_path, folder))]
+	dotfiles = [file for file in dotfiles_for_backup if os.path.isfile(
+		os.path.join(home_path, file))]
+	dotfolders = [folder for folder in dotfolders_for_backup if os.path.exists(
+		os.path.join(home_path, folder))]
 
 	# dotfiles/dotfolders multiprocessing in list format: [(full_dotfile_path, full_dest_path), ...]
 
@@ -179,14 +186,17 @@ def backup_dotfiles(backup_path):
 
 	# Sublime Text Configs
 	if os.path.isdir("/Users/alichtman/Library/Application Support/Sublime\ Text\ 2"):
-		dotfolders_mp_in.append((os.path.join(home_path, "Library/Application\ Support/Sublime\ Text\ 2/Packages/User"), backup_path))
+		dotfolders_mp_in.append((os.path.join(
+			home_path, "Library/Application\ Support/Sublime\ Text\ 2/Packages/User"), backup_path))
 
 	if os.path.isdir("/Users/alichtman/Library/Application Support/Sublime\ Text\ 3"):
-		dotfolders_mp_in.append((os.path.join(home_path, "Library/Application\ Support/Sublime\ Text\ 3/Packages/User"), backup_path))
+		dotfolders_mp_in.append((os.path.join(
+			home_path, "Library/Application\ Support/Sublime\ Text\ 3/Packages/User"), backup_path))
 
 	# XCode Configs
 	if os.path.isdir("/Users/alichtman/Library/Developer/Xcode/UserData"):
-		dotfolders_mp_in.append(("/Users/alichtman/Library/Developer/Xcode/UserData", backup_path))
+		dotfolders_mp_in.append(
+			("/Users/alichtman/Library/Developer/Xcode/UserData", backup_path))
 
 	# pprint(dotfiles_mp_in)
 	# pprint(dotfolders_mp_in)
@@ -222,37 +232,44 @@ def backup_packages(backup_path):
 	for mgr in std_backup_package_managers:
 		# deal with package managers that have spaces in them.
 		print(Fore.BLUE + "Backing up {} package list...".format(mgr) + Style.RESET_ALL)
-		command = "{0} list > {1}/{2}_list.txt".format(mgr, backup_path, mgr.replace(" ", "-"))
+		command = "{0} list > {1}/{2}_list.txt".format(
+			mgr, backup_path, mgr.replace(" ", "-"))
 		sp.run(command, shell=True, stdout=sp.PIPE)
 
 	# cargo
 	print(Fore.BLUE + "Backing up cargo package list..." + Style.RESET_ALL)
-	sp.run("ls {0}/.cargo/bin/ > {1}/cargo_list.txt".format(os.path.expanduser('~'), backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("ls {0}/.cargo/bin/ > {1}/cargo_list.txt".format(os.path.expanduser('~'),
+                                                         backup_path), shell=True, stdout=sp.PIPE)
 
 	# pip
 	print(Fore.BLUE + "Backing up pip package list..." + Style.RESET_ALL)
-	sp.run("pip list --format=freeze > {}/pip_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("pip list --format=freeze > {}/pip_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 
 	# npm
 	print(Fore.BLUE + "Backing up npm package list..." + Style.RESET_ALL)
-	sp.run("npm ls --parseable=true > {}/npm_temp_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("npm ls --parseable=true > {}/npm_temp_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 	# Parse npm output
 	with open("{0}/npm_temp_list.txt".format(backup_path), mode="r+") as f:
 		# Skip first line of file
 		skip = True
-		sp.run("touch {0}/npm_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+		sp.run("touch {0}/npm_list.txt".format(backup_path),
+		       shell=True, stdout=sp.PIPE)
 		with open("{0}/npm_list.txt".format(backup_path), mode="r+") as dest:
 			for line in f:
 				if not skip:
 					dest.write(line.split("/")[-1])
 				skip = False
 
 	# remove temp file
-	sp.run("rm {}/npm_temp_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("rm {}/npm_temp_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 
 	# atom package manager
 	print(Fore.BLUE + "Backing up Atom package list..." + Style.RESET_ALL)
-	sp.run("apm list --installed --bare > {}/apm_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("apm list --installed --bare > {}/apm_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 
 	# sublime text packages
 	if os.path.isdir("/Users/alichtman/Library/Application Support/Sublime Text 2"):
@@ -267,11 +284,13 @@ def backup_packages(backup_path):
 
 	# macports
 	print(Fore.BLUE + "Backing up macports package list..." + Style.RESET_ALL)
-	sp.run("port installed requested > {}/macports_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("port installed requested > {}/macports_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 
 	# system installs
 	print(Fore.BLUE + "Backing up system application list..." + Style.RESET_ALL)
-	sp.run("ls /Applications/ > {}/installed_apps_list.txt".format(backup_path), shell=True, stdout=sp.PIPE)
+	sp.run("ls /Applications/ > {}/installed_apps_list.txt".format(backup_path),
+	       shell=True, stdout=sp.PIPE)
 
 	# Clean up empty package list files
 	print(Fore.BLUE + "Cleaning up empty package lists..." + Style.RESET_ALL)
@@ -328,7 +347,8 @@ def reinstall_packages(packages_path):
 	# construct commands
 	for pm in package_mgrs:
 		if pm in ["brew", "brew-cask"]:
-			cmd = "xargs {0} install < {1}/{2}_list.txt".format(pm.replace("-", " "), packages_path, pm)
+			cmd = "xargs {0} install < {1}/{2}_list.txt".format(
+				pm.replace("-", " "), packages_path, pm)
 			print(cmd)
 			sp.call(cmd, shell=True, stdout=sp.PIPE)
 		elif pm == "npm":
@@ -429,7 +449,8 @@ def cli(complete, dotfiles, packages, fonts, old_path, new_path, reinstall, dele
 
 	# if config file doesn't exist, create it.
 	if not os.path.exists(config_path):
-		print(Fore.BLUE + Style.BRIGHT + "Creating config file at {}".format(config_path))
+		print(Fore.BLUE + Style.BRIGHT +
+		      "Creating config file at {}".format(config_path))
 		config['USER'] = {'backup_path': 'DEFAULT'}
 		with open(config_path, 'w') as f:
 			config.write(f)

constants.py

@@ -1,8 +1,8 @@
 class Constants:
-	PROJECT_NAME='shallow-backup'
-	VERSION='1.3'
-	AUTHOR_GITHUB='alichtman'
-	AUTHOR_FULL_NAME='Aaron Lichtman'
-	DESCRIPTION="Easily create lightweight documentation of installed packages, dotfiles, and more."
-	URL='https://github.com/alichtman/shallow-backup',
-	AUTHOR_EMAIL='aaronlichtman@gmail.com',
+	PROJECT_NAME = 'shallow-backup'
+	VERSION = '1.3'
+	AUTHOR_GITHUB = 'alichtman'
+	AUTHOR_FULL_NAME = 'Aaron Lichtman'
+	DESCRIPTION = "Easily create lightweight documentation of installed packages, dotfiles, and more."
+	URL = 'https://github.com/alichtman/shallow-backup',
+	AUTHOR_EMAIL = 'aaronlichtman@gmail.com',

setup.py

@@ -89,7 +89,7 @@
     # For example, the following would provide a command called `sample` which
     # executes the function `main` from this package when invoked:
     entry_points={
-        'console_scripts':'shallow-backup=shallow_backup:cli'
+        'console_scripts': 'shallow-backup=shallow_backup:cli'
     },
 
     # List additional URLs that are relevant to your project as a dict.