Skip to content
This repository was archived by the owner on Sep 19, 2021. It is now read-only.

Exposing params in run scripts#30

Merged
vaage merged 1 commit intomasterfrom
update-run-scripts
Apr 4, 2017
Merged

Exposing params in run scripts#30
vaage merged 1 commit intomasterfrom
update-run-scripts

Conversation

@vaage
Copy link
Copy Markdown
Contributor

@vaage vaage commented Apr 4, 2017

Before the run script had defaults for most values. This starting causing problems when hosting the server as it would require the run scripts to change.

Now all the parameters are passed in when calling the run script. If any value is missing, the script exits and a usage message is printed.

Before the run script had defaults for most values. This
starting causing problems when hosting the server as it
would require the run scripts to change.

Now all the paramters are passed in when calling the run
script. If any value is missing, the script exits and a
usage message is printed.
@vaage vaage added this to the Relay Server Completion milestone Apr 4, 2017
@vaage vaage self-assigned this Apr 4, 2017
@vaage vaage requested a review from crepricg April 4, 2017 23:00
@vaage vaage merged commit ff2ac9b into master Apr 4, 2017
@vaage vaage deleted the update-run-scripts branch April 4, 2017 23:05
davjd added a commit to davjd/codeu_project_2017 that referenced this pull request Jul 14, 2017
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