Skip to content

Changing Problem::add_output to accept Parameters#892

Merged
andrsd merged 1 commit intomainfrom
output-api-changes
Oct 21, 2025
Merged

Changing Problem::add_output to accept Parameters#892
andrsd merged 1 commit intomainfrom
output-api-changes

Conversation

@andrsd
Copy link
Copy Markdown
Owner

@andrsd andrsd commented Oct 21, 2025

  • it creates the object and stores it
  • it returns back the created object

@andrsd andrsd added the api-change PR changes API, i.e. it will break depending apps label Oct 21, 2025
@andrsd andrsd self-assigned this Oct 21, 2025
- it creates the object and stores it
- it returns back the created object
@andrsd andrsd force-pushed the output-api-changes branch from f378b02 to 2cdf91b Compare October 21, 2025 20:20
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.98%. Comparing base (54a3345) to head (2cdf91b).
⚠️ Report is 7 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #892      +/-   ##
==========================================
- Coverage   74.11%   73.98%   -0.13%     
==========================================
  Files         161      159       -2     
  Lines       12412    12327      -85     
==========================================
- Hits         9199     9120      -79     
+ Misses       3213     3207       -6     

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

@andrsd andrsd merged commit 58b6f49 into main Oct 21, 2025
9 of 10 checks passed
@andrsd andrsd deleted the output-api-changes branch October 21, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api-change PR changes API, i.e. it will break depending apps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant