Allow ForwardDiff v1#1957
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1957 +/- ##
=======================================
Coverage 86.20% 86.20%
=======================================
Files 146 146
Lines 8769 8769
=======================================
Hits 7559 7559
Misses 1210 1210 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
The test errors look real. |
It still doesn't make PRs for dependencies only in |
|
Which is annoying for CI but at least irrelevant for all downstream packages as the compat entries of |
| uuid = "31c24e10-a181-5473-b8eb-7969acd0382f" | ||
| authors = ["JuliaStats"] | ||
| version = "0.25.118" | ||
| version = "0.25.119" |
There was a problem hiding this comment.
The PR doesn't change the package, pnly the tests, so we don't need a new release:
| version = "0.25.119" | |
| version = "0.25.118" |
There was a problem hiding this comment.
I made this because add ForwardDiff@1 leads to "ERROR: Unsatisfiable requirements detected for package Distributions [31c24e10]:", which involved LsqFit... but you may be right that it's not what's bounding ForwardDiff, not really sure.
There was a problem hiding this comment.
Did you encounter it when trying to update a downstream package?
Not sure why CompatHelper didn't make a PR for this.