Skip to content
This repository has been archived by the owner on Dec 4, 2020. It is now read-only.

Adding -h to print usage, just to avoid the error message #236

Merged

Conversation

nickdgriffin
Copy link
Contributor

Previously "./quickstarh.sh -h" would print an error about it being an invalid parameter and print the usage, now it'll just print the usage.

Also means in future it could have an expanded help if necessary.

@nickdgriffin nickdgriffin force-pushed the feature/quickstart_h_flag branch from bdf3c3f to 1ffa6d2 Compare July 20, 2017 12:18
@RobertNorthard
Copy link
Contributor

RobertNorthard commented Jul 29, 2017

LGTM. Thanks @nickdgriffin.

$ ./quickstart.sh -h
 .....
 .....
Usage:
        ./quickstart.sh 
	    -t local 
	    [-m <MACHINE_NAME>] 
	    [-u <INITIAL_ADMIN_USER>] 
	    [-p <INITIAL_ADMIN_PASSWORD>]

        ./quickstart.sh 
	    -t aws 
	    -m <MACHINE_NAME> 
	    -c <AWS_VPC_ID> 
	    -r <AWS_DEFAULT_REGION> 
	    [-z <AVAILABILITY_ZONE_LETTER>] 
	    [-a <AWS_ACCESS_KEY>] 
	    [-s <AWS_SECRET_ACCESS_KEY>] 
	    [-u <INITIAL_ADMIN_USER>] 
	    [-p <INITIAL_ADMIN_PASSWORD>]

@RobertNorthard RobertNorthard merged commit a531789 into Accenture:master Jul 29, 2017
@nickdgriffin nickdgriffin deleted the feature/quickstart_h_flag branch August 1, 2017 12:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants