-
Notifications
You must be signed in to change notification settings - Fork 162
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
CI updates #145
CI updates #145
Conversation
…into development Conflicts: firmware/open_evse/open_evse.h
To ensure that we get the correct versions for each different build, probably a better way to do this...
…t of the build process anyway so removed
…into ci_updates
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, this mostly looks good. I support having a central repository for OpenEVSE controller FW. It has been a source of confusion, for users having multiple openevse FW repos.
platformio.ini
Outdated
upload_flags = "-e -B0.5" | ||
version = 7.1.5 | ||
eu_build_flags = | ||
-D PP_AUTO_AMPACITY |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PP_AUTO_AMPACITY
only applies to emonevse
and emonevse_3ph
environments, NOT openevse_eu
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PP_AUTO_AMPACITY
should be removed from common eu_build_flags
and only applied to emonevse
and emonevse_3ph
environments. Sorry, I can't see how I can make this change myself.
Only apply `PP_AUTO_AMPACITY` to `emonevse`
…into ci_updates
Nice, this all looks good to me now. Although I can't test it fully until it's merged. However, I always compile using platformIO using the same flags, so the build artefacts should be the same 👍 |
Charges to merge various forks and implement CI to build the different configs
Fixes #144