Skip to content

Bob the Building Samples#133

Merged
ActuallyTaylor merged 23 commits intomainfrom
feature/taylor/actions
Mar 23, 2026
Merged

Bob the Building Samples#133
ActuallyTaylor merged 23 commits intomainfrom
feature/taylor/actions

Conversation

@ActuallyTaylor
Copy link
Copy Markdown
Member

@ActuallyTaylor ActuallyTaylor commented Mar 20, 2026

Created a new Create Sample Builds action that creates an artifact of all samples from EVT-Core. This also includes a fix for cmake.yml that actually builds f446 for all PRs. A fix to IWDG on f446 was needed for this. The definition of IWDG was missing from the f446 definitions in manager.hpp.

The Create Sample Builds action produces debug and release.

All Changes

  • Added release.yml GitHub Action that creates debug & release builds of all samples.
  • Re-introduced f446 builds to the main PR GitHub Action (cmake.yml).
  • Added a missing definition in manager.hpp for IWDG on f446.
  • Fixed a comment in IWDGf446.cpp that mentioned F3.

Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Just had to put the exclusion after the grab. I thought it would be a scanning system but its a filtering system instead 🤷‍♀️.

Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Added debug and release builds. This will be useful for templates for other repositories

Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
@ActuallyTaylor ActuallyTaylor requested a review from a team March 20, 2026 02:20
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
it was in the wrong spot within the arguments list since I pasted the CMAKE_BUILD_TYPE argument around.

Signed-off-by: Taylor Lineman <git@actuallytaylor.com>
@ActuallyTaylor ActuallyTaylor changed the title Add a release that uploads built files Bob the Building Samples Mar 20, 2026
@ActuallyTaylor ActuallyTaylor marked this pull request as ready for review March 20, 2026 02:36
Copy link
Copy Markdown
Contributor

@aclowmclaughlin aclowmclaughlin left a comment

Choose a reason for hiding this comment

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

Some minor typo changes.

ActuallyTaylor and others added 4 commits March 21, 2026 10:27
Co-authored-by: Rue <61093955+aclowmclaughlin@users.noreply.github.com>
Co-authored-by: Rue <61093955+aclowmclaughlin@users.noreply.github.com>
Co-authored-by: Rue <61093955+aclowmclaughlin@users.noreply.github.com>
Co-authored-by: Rue <61093955+aclowmclaughlin@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@aclowmclaughlin aclowmclaughlin left a comment

Choose a reason for hiding this comment

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

Approving- btw I don't think there is an issue with setting the permissions to read, I just genuinely didn't know what it did.

@ActuallyTaylor
Copy link
Copy Markdown
Member Author

Approving- btw I don't think there is an issue with setting the permissions to read, I just genuinely didn't know what it did.

I assumed you had no issue with it! After reviewing it, I just realized there's not really a point since the cmake.yml files uses older ways to update code. So it doesn't make a ton of sense for it. Maybe in a future pr we can update it to use modern action conventions!

@DannyCato DannyCato self-requested a review March 22, 2026 19:32
Copy link
Copy Markdown
Contributor

@DannyCato DannyCato left a comment

Choose a reason for hiding this comment

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

I'm kind of confused on the need to build samples in release mode. What would that give us as a benefit?

If we are building binaries, wouldn't we rather them be .elfs instead?

Small questions, but overall looks good to me. Just want to make sure before I approve

@ActuallyTaylor
Copy link
Copy Markdown
Member Author

ActuallyTaylor commented Mar 23, 2026

@DannyCato

I'm kind of confused on the need to build samples in release mode. What would that give us as a benefit?

Building in release mode is not strictly needed for samples. To be honest, its not needed at all. But, the point of this PR was to build a backbone for future GitHub actions on all other repositories to build debug and release builds of targets.

If we are building binaries, wouldn't we rather them be .elfs instead?

.elfs would be useful for our analysis, but we never flash them onto boards. By providing the built .bin files we can flash pieces of a bike by just downloading the last working releases from GitHub. This will make race day flashing a lot easier, and reduce the need for firmware team members to build projects for EEs or Integration when they just need to quickly validate a board.

Copy link
Copy Markdown
Contributor

@DannyCato DannyCato left a comment

Choose a reason for hiding this comment

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

I like those reasons

@ActuallyTaylor ActuallyTaylor merged commit c4cdf81 into main Mar 23, 2026
2 checks passed
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.

3 participants