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

actionified and formatted - needs doxygenation #1

Merged
merged 7 commits into from
Apr 12, 2020
Merged

Conversation

siddacious
Copy link
Contributor

@siddacious siddacious commented Apr 8, 2020

This PR adds the basic stuff for GitHub actions and a start to addressing the formatting and documentation requirements.

TODO:
The documentation needs to be converted to a Doxygen compatible format
I'm guessing the build will fail because LittlevGL is required to compile the examples and it's not available though the Arduino library manager. We'll have to figure out how we want to handle this

  • Examples may need more attention

You'll see there is a rather large amount of dependencies listed now; by default the actions workflow will attempt to compile each example on a number of platforms specified here:
https://github.com/adafruit/Adafruit_LvGL_Glue/blob/actionci/.github/workflows/githubci.yml#L23
which refers to this file:
https://github.com/adafruit/ci-arduino/blob/master/build_platform.py#L66

There is a mechanism blacklist or whitelist an example for a given platform should that help.

Let me now if you have any questions

@PaintYourDragon
Copy link
Contributor

I think LittlevGL is in the Library manager as lv_arduino

run: bash ci/actions_install.sh

- name: test platforms
run: python3 ci/build_platform.py main_platforms
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This line will need to be updated to change the platforms that are built, drawing from here:
https://github.com/adafruit/ci-arduino/blob/master/build_platform.py#L28

@siddacious
Copy link
Contributor Author

siddacious commented Apr 10, 2020

@siddacious
Copy link
Contributor Author

@PaintYourDragon I originally had it building the hello_gizmo and widgets_gizmo examples for the CPX as well but it looks like it ran out of memory:
https://github.com/adafruit/Adafruit_LvGL_Glue/pull/1/checks?check_run_id=577457962#step:6:90

I thought I heard @ladyada mention having a simple UNO example as well so I originally included it as well and had it only build the featherwing examples but it had the same issue obvs.

@PaintYourDragon PaintYourDragon merged commit ab7883d into master Apr 12, 2020
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.

2 participants