Merge pull request #41 from h4rldev/h4rl-patch-1

Clean up repo kind of?

Author: charlie-sans

addons/Notepad.py

@@ -6,9 +6,9 @@
 #-----------------------------------------#
 #this notepad applacation was made by six519
 # https://github.com/six519/Python-Notepad
-# this program plugin change applys the origonal creators repos licence.md file
-# any addons added to PythonicOS shall make PythonicOS free and open source
-#
+# This program plugin change applies the original creators repos license.md file
+# Any addons added to PythonicOS shall make PythonicOS free and open source
+# Great that you actually credited the original creator on this one :) -h4rl
 #
 #-----------------------------------------#
 

addons/__pycache__/panno.cpython-310.pyc

@@ -1,10 +1,10 @@
 import tkinter as tk
 #-----------------------------------------#
-#this notepad applacation was made by six519
-# https://github.com/six519/Python-Notepad
-# this program plugin change applys the origonal creators repos licence.md file
-# any addons added to desktopy shall make desktopy free and open source
 #
+# 
+# Who made this? - H4rl
+# 
+# 
 #
 #-----------------------------------------#
 

addons/__pycache__/panno.cpython-311.pyc

@@ -6,3 +6,4 @@ option2 = Value2
 option3 = Value3
 option4 = Value4
 
+# If you want a good way to configure, use .conf, yaml, json or toml. Thx -h4rl

addons/__pycache__/panno.cpython-39.pyc

@@ -1,5 +1,5 @@
 import pygame
-
+# Won't this show the annoying pygame prompt though?
 def play_music(music_file):
     """
     stream music with mixer.music module in blocking manner

addons/procsss.py

@@ -149,10 +149,7 @@ def load_files(home_dir):
             grid_column = 0
             grid_row += 1
 
-        def open_file_with_path(path):
-            open_file(path)
-
-        label.bind("<Button-1>", lambda event, path=file_path: open_file_with_path(path))
+        label.bind("<Button-1>", lambda event, path=file_path: open_file(path))
         label.bind("<Button-3>", lambda event, path=file_path: show_files_context_menu(event))
         label.grid(row=grid_row, column=grid_column, padx=10, pady=10, sticky='w')
 

addons/tkinter-calc.py

@@ -1,6 +1,8 @@
 import tkinter as tk
 import tkinter.font as tkFont
-error = 
+error = ""
+
+# How is this supposed to work?
 class App:
     def __init__(self, root):
         #setting title

config.ini

@@ -5,6 +5,8 @@
 home_dir = os.path.join(os.path.dirname(__file__), 'home')
 pythonOS = os.path.join(os.path.dirname(__file__), 'home')
 
+# You should really structure this better and using python for this type of project isn't wise
+
 panno = 'addons/panno.py'
 pyle = 'addons/pyle.py'
 addons = 'addons'

home/midiplayer.py

@@ -1,5 +1,5 @@
 import curses
-
+# ??
 def draw_desktop(stdscr):
     stdscr.clear()
     stdscr.addstr(0, 0, "Desktop")

homme/main.py

@@ -1,11 +1,11 @@
 # loginscreen.py
+# seems unsafe - h4rl
 import subprocess
 import tkinter as tk
 import configparser
 import os
 import argparse
 import configparser
-import file
 from file import usrpass,config_login,shell
 root = tk.Tk()
 

main.spec

@@ -32,12 +32,12 @@ def startSH():
     print_slow('##           ##       ##    ##     ##  #######  ##    ## ####  ######                #######   ######  ')
     print_slow('*******************************************************************************************************')
     print_slow('-------------------------------------------PythonicOS--------------------------------------------------')
-    print_slow('-------------------------------------------version 1.0-------------------------------------------------')
-    print_slow('-----------------------copywrite 2023 OpenStudio and the amazing github suporters----------------------')
-    print_slow('------------------welcome to the interactive start wizard for Windows, MacOS and Linux.----------------')
+    print_slow('---------------------------------------------v 0.1-----------------------------------------------------')
+    print_slow('------------------Copyright 2023 OpenStudio and our github supporters and collaborators----------------')
+    print_slow('------------------Welcome to the interactive start wizard for Windows, MacOS and Linux.----------------')
     print_slow('-------------------------------------------------------------------------------------------------------')
-    print_slow('-------------------this wizard will allow you to navagate PythonicOS and its features!-----------------')
-    print_slow('---------------using the helpfull commands listed below you can nvagate around the shell!--------------')
+    print_slow('-------------------This wizard will allow you to navagate PythonicOS and its features!-----------------')
+    print_slow('----------------using the helpful commands listed below you can navigate around the shell!-------------')
     print_slow('-------------------------------------------------------------------------------------------------------')
     print_slow('-------------------------------------------------------------------------------------------------------')
     print_slow('-------------------------------------------------------------------------------------------------------')

python.spec

@@ -10,6 +10,7 @@
 config.set('Section2', 'Option3', 'Value3')
 config.set('Section2', 'Option4', 'Value4')
 
+# INI is not really recommended for configuration
 with open('config.ini', 'w') as configfile:
     config.write(configfile)
     print('File created')

python/PythonicOS.py

@@ -1,6 +1,4 @@
 import os
-import sys
-import datetime
 
 DIRS = [
     'home',
@@ -16,7 +14,7 @@
     'system/logs'
     'mount'
 ]
-
+# Why are there different files that create directories?
 def main():
     for i in DIRS:
         if not os.path.exists(i):