Add more and document existing environment variables #479

Merged
merged 5 commits into from Sep 14, 2016

Conversation

Projects
None yet
3 participants
@jougs
Contributor

jougs commented Sep 14, 2016

This is a replacement for #418, which adds the environment variables SLI_PATH for adding paths to the SLI search path and NEST_MODULES for loading modules on startup on the SLI level. It also adds a new documentation file that describes all environment variables that NEST reads.

I propose @obreitwi, @heplesser and @apeyser as reviewers.

+ installations or to accomodate different machine setups.
+
+`SLI_PATH`
+ : A colon separated list of paths, in which NEST searches for SLI

This comment has been minimized.

@apeyser

apeyser Sep 14, 2016

Contributor

paths, in which --> paths which

@apeyser

apeyser Sep 14, 2016

Contributor

paths, in which --> paths which

+also respects the following standard POSIX shell variables:
+
+`COLUMNS`
+ : A number of columns, to which NEST adapt the width of its textual

This comment has been minimized.

@apeyser

apeyser Sep 14, 2016

Contributor

columns, to which -> columns to which

@apeyser

apeyser Sep 14, 2016

Contributor

columns, to which -> columns to which

@apeyser

This comment has been minimized.

Show comment
Hide comment
@apeyser

apeyser Sep 14, 2016

Contributor

Some English lint, looks like it does the same as #418.
👍

Contributor

apeyser commented Sep 14, 2016

Some English lint, looks like it does the same as #418.
👍

@obreitwi

This comment has been minimized.

Show comment
Hide comment
@obreitwi

obreitwi Sep 14, 2016

Looks good to me. Thank you!

On Wed, Sep 14, 2016 at 4:58 PM, Alexander Peyser notifications@github.com
wrote:

Some English lint, looks like it does the same as #418
#418.
👍


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#479 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHhBMIeXuIcCFavWsIoFJ29ee5rac2Cks5qqAt7gaJpZM4J8pou
.

Looks good to me. Thank you!

On Wed, Sep 14, 2016 at 4:58 PM, Alexander Peyser notifications@github.com
wrote:

Some English lint, looks like it does the same as #418
#418.
👍


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#479 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHhBMIeXuIcCFavWsIoFJ29ee5rac2Cks5qqAt7gaJpZM4J8pou
.

@jougs

This comment has been minimized.

Show comment
Hide comment
@jougs

jougs Sep 14, 2016

Contributor

@apeyser: I've fixed the English.
@obreitwi: thanks!

Contributor

jougs commented Sep 14, 2016

@apeyser: I've fixed the English.
@obreitwi: thanks!

@apeyser apeyser merged commit 96784ce into nest:master Sep 14, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment