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

CI Setup #6

Merged
merged 3 commits into from
Aug 4, 2021
Merged

CI Setup #6

merged 3 commits into from
Aug 4, 2021

Conversation

sudotensor
Copy link
Owner

Please refer to the associates issue (#4) for more information. In summary, a GitHub actions workflow was setup with jobs to build both the Mock and MCUboot examples. For now, the workflow only tests if the examples build without errors. For the the MCUboot example, "yes" is piped into the script to automate the version number change in mbed_app.json using the jq cli tool.

Note: It is not a strict requirement to test that these examples work on the target board through the workflow. However, this could be done at a later point through a separate workflow.

Closes #4

@sudotensor sudotensor added the enhancement New feature or request label Aug 4, 2021
@sudotensor sudotensor self-assigned this Aug 4, 2021
@sudotensor sudotensor merged commit 9779527 into main Aug 4, 2021
@sudotensor sudotensor deleted the ci-setup branch August 4, 2021 10:24
@sudotensor sudotensor restored the ci-setup branch August 4, 2021 10:32
@sudotensor sudotensor mentioned this pull request Aug 4, 2021
@sudotensor sudotensor deleted the ci-setup branch August 4, 2021 10:34
@sudotensor sudotensor restored the ci-setup branch August 4, 2021 10:36
@sudotensor
Copy link
Owner Author

sudotensor commented Aug 4, 2021

There was a small hiccup with this pull request. The request was merged before the workflow completed its run (and failed). I realised that the workflow was missing a critical dependency - the GCC ARM toolchain. main has been reset to its state before the pull request was merged through yet another pull request #7.

Update: The issue has been fixed through #8. ✅

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

Successfully merging this pull request may close these issues.

Setup CI workflow
1 participant