Skip to content

Small cleanup - #2505

Merged
jajhall merged 5 commits into
ERGO-Code:latestfrom
fwesselm:removeDuplicateLogic
Aug 19, 2025
Merged

Small cleanup#2505
jajhall merged 5 commits into
ERGO-Code:latestfrom
fwesselm:removeDuplicateLogic

Conversation

@fwesselm

Copy link
Copy Markdown
Collaborator
  • Small code cleanup.
  • Code changes do not have any effect on HiGHS behavior.

@fwesselm
fwesselm requested a review from Opt-Mucca August 18, 2025 08:36

@jajhall jajhall left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fine so long as you're sure the changes don't change execution

@codecov

codecov Bot commented Aug 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (latest@f6255b8). Learn more about missing BASE report.
⚠️ Report is 9 commits behind head on latest.

Additional details and impacted files
@@            Coverage Diff            @@
##             latest    #2505   +/-   ##
=========================================
  Coverage          ?   79.72%           
=========================================
  Files             ?      346           
  Lines             ?    85939           
  Branches          ?        0           
=========================================
  Hits              ?    68519           
  Misses            ?    17420           
  Partials          ?        0           

☔ 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.

@Opt-Mucca Opt-Mucca left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good!

@jajhall
jajhall merged commit 5d45136 into ERGO-Code:latest Aug 19, 2025
213 of 214 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