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

[cbuild2cmake] Set cmse-lib as a CMake custom command byproduct #261

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

brondani
Copy link
Collaborator

@brondani brondani commented Feb 7, 2025

Fixes

Changes

  • Set the cmse-lib as a custom build event byproduct, re-triggering the linker step whenever the cmse-lib is missing for whatever reason.

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

Copy link
Contributor

github-actions bot commented Feb 7, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Files

Copy link
Contributor

github-actions bot commented Feb 7, 2025

Test Results

  4 files  ±0   20 suites  ±0   1s ⏱️ -1s
 74 tests ±0   74 ✅ ±0  0 💤 ±0  0 ❌ ±0 
296 runs  ±0  296 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit b263ea0. ± Comparison against base commit 43ff5d6.

Copy link

codeclimate bot commented Feb 7, 2025

Code Climate has analyzed commit b263ea0 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 88.1% (0.0% change).

View more on Code Climate.

@brondani brondani merged commit 29836b8 into main Feb 7, 2025
17 checks passed
@brondani brondani deleted the cbuild2cmake-cmse-lib-custom-byproduct branch February 7, 2025 10:33
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