Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: expose some config options as env variables #1099

Merged
merged 2 commits into from
Mar 13, 2022

Conversation

mKeRix
Copy link
Owner

@mKeRix mKeRix commented Mar 10, 2022

Describe the change

Adds custom environment variable mappings for use by node-config. For now, it only includes a subset of options as suggested in the issue which triggered this PR.

Note: I haven't tested these changes with a real installation yet, just on my dev machine. Will merge after I had time to do real world tests.

Checklist
If you changed code:

  • Tests run locally and pass (npm test)
  • Code has correct format (npm run format)

If you added a new integration:

  • Documentation page added in docs/integrations/
  • Page linked in docs/.vuepress/config.ts and docs/integrations/README.md

Additional information
Closes #1094.

@mKeRix mKeRix linked an issue Mar 10, 2022 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Mar 10, 2022

Codecov Report

Merging #1099 (57d6a9c) into main (7eb8b43) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1099   +/-   ##
=======================================
  Coverage   91.39%   91.39%           
=======================================
  Files          97       97           
  Lines        2708     2708           
  Branches      463      463           
=======================================
  Hits         2475     2475           
  Misses        223      223           
  Partials       10       10           
Flag Coverage Δ
unittests 91.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ee057d8...57d6a9c. Read the comment docs.

@mKeRix
Copy link
Owner Author

mKeRix commented Mar 13, 2022

Confirmed it's working on a real installation.

@mKeRix mKeRix merged commit 3ae8e9a into main Mar 13, 2022
@mKeRix mKeRix deleted the 1094-configuration-via-environment-variables branch March 13, 2022 20:03
github-actions bot pushed a commit that referenced this pull request Mar 20, 2022
# [2.20.0](v2.19.1...v2.20.0) (2022-03-20)

### Bug Fixes

* **bluetooth-classic:** increase change detection speed ([6325937](6325937)), closes [#775](#775)
* **docker:** include heatmap dependencies in image ([7cf5de1](7cf5de1)), closes [#826](#826)

### Features

* **bluetooth:** add configuration options for noise filter ([85981ee](85981ee)), closes [#775](#775)
* expose some config options as env variables ([#1099](#1099)) ([3ae8e9a](3ae8e9a)), closes [#1094](#1094)
@github-actions
Copy link

🎉 This PR is included in version 2.20.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Mar 20, 2022
# [3.0.0-beta.5](v3.0.0-beta.4...v3.0.0-beta.5) (2022-03-20)

### Bug Fixes

* **bluetooth-classic:** increase change detection speed ([6325937](6325937)), closes [#775](#775)
* **docker:** include heatmap dependencies in image ([7cf5de1](7cf5de1)), closes [#826](#826)

### Features

* **bluetooth:** add configuration options for noise filter ([85981ee](85981ee)), closes [#775](#775)
* expose some config options as env variables ([#1099](#1099)) ([3ae8e9a](3ae8e9a)), closes [#1094](#1094)
@github-actions
Copy link

🎉 This PR is included in version 3.0.0-beta.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

Configuration via environment variables
1 participant