-
Notifications
You must be signed in to change notification settings - Fork 148
Read params from src/config.h. Precedence given to platformio config. #204
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
base: master
Are you sure you want to change the base?
Read params from src/config.h. Precedence given to platformio config. #204
Conversation
|
This PR is marked as stale because it has been open 45 days with no activity. You can remove stale label or comment if this PR is still valid. |
|
Happy to take suggestions on how to improve this PR. Please advise! |
|
This PR is marked as stale because it has been open 45 days with no activity. You can remove stale label or comment if this PR is still valid. |
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
|
This PR is marked as stale because it has been open 45 days with no activity. You can remove stale label or comment if this PR is still valid. |
4e24f8a to
7e03523
Compare
|
This PR is marked as stale because it has been open 45 days with no activity. You can remove stale label or comment if this PR is still valid. |
Many projects use src/config.h to define hostname, username, and password constants in C++ outside of version control using .gitignore. Not sure if it's in fitting with this project's aesthetic to be (admittedly) hackishly pulling from src/config.h if it exists, but this is a huge convenience and makes it easier to keep credentials in a single, secure place. Happy to take suggestions on improvements.