Skip to content

Conversation

@franckgaga
Copy link
Member

Hi @baggepinnen,

Do you have experience with package extension? I tried to add a simple package extension LinearMPCext in this PR, and no matter what, I'm not able to call the dummy hi function. Do you spot an obvious problem? I used ChatGPT and Claude to revise the file structure and my Project.toml and they were not helpful at all.

The goal here would be to load the code generation tools only if LinearMPC is available and loaded.

Many thanks!

@franckgaga franckgaga marked this pull request as draft December 10, 2025 21:50
@franckgaga
Copy link
Member Author

Wait a minute, it's not clear for you to review the PR since the base commit was a super old commit. I will close and open a cleaner one.

@franckgaga franckgaga closed this Dec 10, 2025
@franckgaga franckgaga deleted the extension branch December 10, 2025 22:34
@codecov-commenter
Copy link

codecov-commenter commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 98.53%. Comparing base (399968e) to head (6501790).

Files with missing lines Patch % Lines
ext/LinearMPCext.jl 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #296      +/-   ##
==========================================
- Coverage   98.55%   98.53%   -0.02%     
==========================================
  Files          28       29       +1     
  Lines        4967     4968       +1     
==========================================
  Hits         4895     4895              
- Misses         72       73       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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