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.

Sign up for GitHub

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

Jump to bottom

Cleaned up the files and changed the installation script #2

Merged

Gcat101 merged 3 commits into rainbow-sh:master from mina-atef-00:master

Dec 26, 2021

Contributor

mina-atef-00 commented Dec 26, 2021

Fixed a typo in the README. 80298c1
Added visual studio code and python files to ./.gitignore to remove the cache files. 88e8f98

Installation Script Changes: 3099416

Requires running with sudo using sudo bash install.sh to copy to /usr/local/bin/
Exits on command errors showing which command exited with error.
New Script Output (doesn't show pip output):

❯ sudo bash "install.sh"
[sudo] password for pure-cheekbones: 
Installing package requirements...
Installing package...
Cleaning up...
Package Installed! 

Please add this line to your shell profile (You are using /bin/bash):
'export WEATHER_CLI_API=<your OWM api key>'

mina-atef-00 added 3 commits

December 26, 2021 17:01


          typo fix

80298c1


          removed cache files, modified gitignore

88e8f98


          added error handling, beautified script output

Member

Gcat101 commented Dec 26, 2021

Seems good to me, thanks for your contribution.

Gcat101 merged commit d6eb40f into rainbow-sh:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment