fix - make instructions more clear in the README #31
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
RobertLucian
added a commit
that referenced
this pull request
May 9, 2018
… old one (#30) * couple of new functions * use $HOME, and do shallow clone * It seems we need sudo here * create folder withhout sudo * install setup.py package locally * Fix the adding and removal of control panels on boot * Move iteration inside of remove_control_panel so we only delete once * feature - arg based installation (#26) * feature - add arg parser + change how things get installed * feature - condense code into just one line of code * feature - add show-hows to README * feature - remove artifacts in README * feature - add python3 to the table too * feature - treat situation for python3 executable * feature - fix syntax issues with installer script * feature - fix syntax error caused by missing fi keyword * feature - fix semantics errors and add some error handlers * feature - make the installation of the python package optional * feature - add correct install command for basic example in README * feature - add check_internet function to list of functions * feature - add message logs for console * feature - fix small syntax error * feature - add missing packages that are mandatory * feature - install git before git cloning if it's not present * feature - add exit code 0 at the end of the installer script * feature - make sure Dexter is owned by pi * feature - add some comments for delimiting the script into sections * feature - add nodejs repo to apt-get sources * feature - use 3rd person instead of 1st * feature - reorganize script_tools the same way as the grovepi * feature - fix bugs with install script * feature - couple of output-related fixes * feature - add missing logic into install script * feature - pass arguments to function in install script * feature - check if script is run with pi user * feature - correct name of installer * feature - add pip/pip3 as deb packages * feature - add comment & change mkdir commands as Nicole suggested * fix - specify the right way to curl the install script (#29) * fix - make instructions more clear in the README (#31) * Update README.md * Update README.md * fix - remove "-s" option from piped bash
RobertLucian
added a commit
that referenced
this pull request
Jun 1, 2018
* couple of new functions * use $HOME, and do shallow clone * It seems we need sudo here * create folder withhout sudo * install setup.py package locally * Fix the adding and removal of control panels on boot * Move iteration inside of remove_control_panel so we only delete once * Make create_folder use a multi-level path if needed * feature - arg based installation (#26) * feature - add arg parser + change how things get installed * feature - condense code into just one line of code * feature - add show-hows to README * feature - remove artifacts in README * feature - add python3 to the table too * feature - treat situation for python3 executable * feature - fix syntax issues with installer script * feature - fix syntax error caused by missing fi keyword * feature - fix semantics errors and add some error handlers * feature - make the installation of the python package optional * feature - add correct install command for basic example in README * feature - add check_internet function to list of functions * feature - add message logs for console * feature - fix small syntax error * feature - add missing packages that are mandatory * feature - install git before git cloning if it's not present * feature - add exit code 0 at the end of the installer script * feature - make sure Dexter is owned by pi * feature - add some comments for delimiting the script into sections * feature - add nodejs repo to apt-get sources * feature - use 3rd person instead of 1st * feature - reorganize script_tools the same way as the grovepi * feature - fix bugs with install script * feature - couple of output-related fixes * feature - add missing logic into install script * feature - pass arguments to function in install script * feature - check if script is run with pi user * feature - correct name of installer * feature - add pip/pip3 as deb packages * feature - add comment & change mkdir commands as Nicole suggested * fix - specify the right way to curl the install script (#29) * fix - make instructions more clear in the README (#31) * Update README.md * Update README.md * fix - remove "-s" option from piped bash * minor changes * feature - strip down script_tools (#33) * feature - strip down script_tools to the bare minimum * feature - pass on arguments to parser function * feature - remove unneeded sentence from output * feature - use develop branch of DexterInd at all times * Add a sudo_add_to_end_of_file function * feature - use dynamic selection of branch (#34)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.