CompatHelper: bump compat for RecursiveArrayTools to 4, (keep existing compat)#291
Conversation
5fff896 to
01712ff
Compare
Benchmark Results (Julia v1.10)Time benchmarks
Memory benchmarks
|
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Coverage Report for CI Build 25158120313Coverage remained the same at 98.483%Details
Uncovered Changes
Coverage RegressionsNo coverage regressions found. Coverage Stats
💛 - Coveralls |
ranocha
left a comment
There was a problem hiding this comment.
It still uses the old version in CI: https://github.com/NumericalMathematics/DispersiveShallowWater.jl/actions/runs/23974323780/job/69928886531?pr=291#step:7:182
JoshuaLampert
left a comment
There was a problem hiding this comment.
Looks good! The warnings like
┌ Warning: Runtime invalidation was disabled, but the CPU info is out-of-date.
│ Will continue with incorrect CPU name (from build time).
└ @ HostCPUFeatures ~/.julia/packages/HostCPUFeatures/ZTXz4/src/cpu_info.jl:71
were also already present in the last CI run on main: https://github.com/NumericalMathematics/DispersiveShallowWater.jl/actions/runs/24400348279/job/71268950948#step:7:460.
|
@ranocha, you already use dependabot for the julia compat updates in SummationByPartsOperators.jl, don't you? It supports grouping compat PRs right? This makes these kind of compat updates, which depend on each other, easier to handle. Did you make good experiences with dependabot for julia compat PR? Do you think it's time to switch from CompatHelper to dependabot (dependabot for julia is out of beta state already). |
This pull request changes the compat entry for the
RecursiveArrayToolspackage from3.27.2to3.27.2, 4.This keeps the compat entries for earlier versions.
Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.
Closes #292, closes #294, closes #295, closes #296, closes #297, closes #298, closes #299, closes #300, closes #301, closes #302, closes #303.