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

Add CI configuration for most popular firmwares, and fix PlatformIO build recipes #49

Merged
merged 6 commits into from
May 24, 2023

Conversation

amotl
Copy link
Member

@amotl amotl commented May 23, 2023

Hi there,

in order to improve the quality of our most popular firmwares hosted in this repository, this patch brings a few PlatformIO configuration files up to speed, and adds a CI configuration using GitHub Actions.

The most important topics are:

With those adjustments, the firmwares scale-adjust/ADS1231, scale-adjust/HX711, node-gprs-http, node-wifi-mqtt, node-esp8266-generic, and node-esp32-generic, build successfully on CI.

image

Let me know if you have any further suggestions on this topic.

With kind regards,
Andreas.

/cc @msweef

@amotl amotl merged commit 805a9e3 into main May 24, 2023
@amotl amotl deleted the amo/add-ci branch May 24, 2023 08:36
Comment on lines 23 to 31
lib_deps =
vshymanskyy/TinyGSM@^0.10.9
robtillaart/RunningMedian@^0.3.3
robtillaart/[email protected].7
paulstoffregen/OneWire@^2.3.5
bogde/HX711@^0.7.4
bogde/[email protected]
milesburton/DallasTemperature@^3.9.1
bblanchon/ArduinoJson@^6
https://github.com/tzapu/WiFiManager.git#a83ac6e
https://github.com/tzapu/WiFiManager.git#2.0.4-beta
https://github.com/daq-tools/Adafruit_MQTT_Library#maxbuffersize-2048
Copy link
Member Author

@amotl amotl May 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hallo @u-l-m-i,

ich habe den Patch integriert, ohne auf Deine Antwort zu warten, damit Karsten eine gute Basis hat, um sich eine neue Firmware erschließen zu können 1.

Ich gehe nicht davon aus, dass die Aktualiserung der platformio.ini Dateien irgendetwas kaputt gemacht hat, aber ich will es natürlich nicht stillschweigend geschehen lassen, gerade bei Deinen guten best-of-breed Stücken ;].

  • Auf robtillaart/[email protected] zu gehen, ist glaube ich safe.
  • Explizit bogde/[email protected] zu benutzen, ist glaube ich eine gute Sache.
  • tzapu/WiFiManager.git#2.0.4-beta habe ich ausgewählt, weil es zu einer ähnlichen Zeit wie Dein referenzierter a83ac6e auch dieses Release gab. Sag bitte Bescheid, wenn Du denkst, dass ich hier einen Fehler gemacht habe.

Viele Grüße,
Andreas.

Footnotes

  1. https://community.hiveeyes.org/t/dht-is-dead-hx711-ist-dead/4936

platform = espressif32
platform = espressif32@^2
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auch hier habe ich eine Version festgelegt. Die Version 2 1 des platformio/espressif32 Pakets kam so vor 2-3 Jahren raus. Ich denke das ist die Zeit, in der diese Firmware entwickelt wurde?

Es wäre natürlich schön, das auch mit neueren Versionen zu versuchen, aber das würde ich natürlich Dir überlassen, weil es da durchaus Überraschungen zur Laufzeit geben kann, selbst wenn das Bauen klappt.

Footnotes

  1. https://registry.platformio.org/platforms/platformio/espressif32/versions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant