README.md

@@ -1,27 +1,12 @@
 # skratch
 
-# Skratch
+A cli program to create and edit temporary files
 
-A cli program to quickly create and edit temporary files
+Installation:
 
-Requirements:
-python3 > 3.6
-
-## Installation:
-
-    mv skratch.py /usr/local/bin/skratch
-    chmod +x /usr/local/bin/skratch
+mv skratch.py /usr/bin/skratch
 
 set an env variable for $EDITOR and/or $VISUAL in .bashrc
 
 export EDITOR=vi
 export VISUAL=geany
-
-
-
-## Usage
-`skratch` creates a random file in .skratch and opens it in the EDITOR
-`-c` clears/deletes all files in .skratch
-`-l` lists files in .skratch
-`-o <filename>` opens a specific file
-`-v` opens the VISUAL editor

skratch.py

@@ -11,29 +11,20 @@ def make_dir(d):
 		os.mkdir(d)
 		
 def delete_all(d):
-	files = list_all(d)
-	if not files:
-		return
-	x = input("Delete the above files? y/N ")
-	if x.lower() != 'y':
-		return
-	for file in files:
-		f = os.path.join(d,file)
-		os.remove(f)
-		print(f"removing: {f}")
+	if not os.path.exists(d):
+		print("Nothing to delete")
+	else:
+		for file in os.listdir(d):
+			os.remove(d+file)
+			print(f"removing: {d+file}")
 		print("Done")
 		
 def list_all(d):
 	if not os.path.exists(d):
-		print("Skratch path doesn't exist")
-		return 0
-	files = os.listdir(d)
-	if files == []:
-		print("No skratch files exist")
+		print("Nothing to list")
 	else:
-		for file in files:
+		for file in os.listdir(d):
 			print(file)
-	return files
 	
 def get_editor(use_v):
 	v = os.getenv("VISUAL")
@@ -45,9 +36,7 @@ def get_editor(use_v):
 	return e
 
 def mkstemp(fp):
-	if fp[-6:] != "XXXXXX":
-		print(f'Error: mkstemp got filepath: {fp}')
-		return None
+	if fp[-6:] == "XXXXXX":
 		for x in range(0,1000):
 			r = ''.join(random.choices(string.ascii_letters, k=6))
 			fp = fp[:-6] + r
@@ -55,7 +44,9 @@ def mkstemp(fp):
 				return fp
 		print("Error: Could not create new file, try running skratch -c and try again")
 		return None
-		
+	else:
+		print(f'Error: mkstemp got filepath: {fp}')
+		return None
 		
 def run(editor, sk_path, filename=None):
 	if not filename:
@@ -89,7 +80,6 @@ def main():
 	
 	group.add_argument(
 		"-o", 
-		metavar="<file>",
 		help="open file", 
 		action="store")