Skip to content

Conversation

@mkucharz
Copy link
Member

@mkucharz mkucharz commented Feb 16, 2018

If socket name is:
test-123
and it has config option:
API-KEY
CLI is going to look for:
TEST_123__API_KEY env variable and for
TEST_123_API_KEY (for backward compatibility)

@codecov-io
Copy link

codecov-io commented Feb 16, 2018

Codecov Report

Merging #155 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #155      +/-   ##
==========================================
- Coverage   28.32%   28.31%   -0.02%     
==========================================
  Files         102      102              
  Lines        2803     2804       +1     
==========================================
  Hits          794      794              
- Misses       2009     2010       +1

@mkucharz
Copy link
Member Author

Fix #137

@mkucharz mkucharz merged commit c35a41a into master Feb 16, 2018
@mkucharz mkucharz deleted the fix/getting_config_from_vars branch February 16, 2018 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants