-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Some improvement #2
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few points to fix, if you want I can let you commit to a new branch, and I fix them there. thanks for your contributions to my repository!
aa.py
Outdated
@@ -288,57 +288,70 @@ def ResetBash(): | |||
#####AA Zone##### | |||
''' | |||
|
|||
|
|||
def DetectShell(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's check if its none as well, and return bash and zsh rather than .bashrc and .zshrc
except IOError: | ||
print("I Think Bashrc Is Write Protect, Fix It And Run Code Again") | ||
print("I Think Shell Is Write Protect, Fix It And Run Code Again") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Run this again, is better than run code
upgradeit="2.Down/Up-grade AA" | ||
otherver="If You Want to Upgrade Or Find It Useless, This Script Contain a diffrent ver" | ||
olderver="But It's Older :(" | ||
newerver="Good News, It's New :)" | ||
resetshell="To Effect Changes You Need To Run A New Bash Or Source Bashrc, Do You Want To Source Bashrc? (If You Are Already Running This In Bash Its Recommended) (Yes/No) : " | ||
resetshell="To Effect Changes You Need To Run A New Bash Or Source Shell, Do You Want To Source Shell? (If You Are Already Running This In Bash Its Recommended) (Yes/No) : " |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a new bash or source shell XD? lets go with "a new shell or source your shell dotfile", same next line
added zsh support and a function to install some tools like omz and omb