Skip to content

Drop support for Optimization v3#227

Closed
sethaxen wants to merge 5 commits intomainfrom
optimizationv4
Closed

Drop support for Optimization v3#227
sethaxen wants to merge 5 commits intomainfrom
optimizationv4

Conversation

@sethaxen
Copy link
Member

The constructor for OptimizationCache changed between Optimization v3 and v4. #218 constructs such a cache, so this PR drops support for Optimization v3. To ensure our compat lower bounds are mutually compatible (tested by the downgrade versions of actions), this PR also bumps the lower bounds for many package, test, and docs dependencies.

@codecov
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.16%. Comparing base (52fef88) to head (b8b325d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #227   +/-   ##
=======================================
  Coverage   83.16%   83.16%           
=======================================
  Files          13       13           
  Lines         588      588           
=======================================
  Hits          489      489           
  Misses         99       99           

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

@sethaxen
Copy link
Member Author

In #218 I was able to find a workaround to keep support for Optimization >= v3.24. Since bumping to a lower bound of v4 causes so many version restrictions across the board, that seems safer for now.

@sethaxen sethaxen closed this Jan 16, 2025
@sethaxen sethaxen deleted the optimizationv4 branch January 16, 2025 12:59
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.

1 participant