search

EnzymeAD / Enzyme.jl

Julia bindings for the Enzyme automatic differentiator
https://enzyme.mit.edu
MIT License
422 stars 58 forks source link

Update `CompatHelper` workflow #1478

Closed prbzrg closed 4 weeks ago

prbzrg commented 1 month ago

I added other directories to check by CompatHelper. Also, I enabled the check for jlls. Before merging this, you should know that at first, it will create many PRs.

codecov-commenter commented 4 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 74.77%. Comparing base (74d88b4) to head (8c2f5ac). Report is 13 commits behind head on main.

:exclamation: Current head 8c2f5ac differs from pull request most recent head 6a03a0c

Please upload reports for the commit 6a03a0c to get more accurate results.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1478 +/- ## ========================================== + Coverage 73.80% 74.77% +0.96% ========================================== Files 30 30 Lines 11252 11391 +139 ========================================== + Hits 8305 8518 +213 + Misses 2947 2873 -74 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

wsmoses commented 4 weeks ago

@vchuravy @prbzrg can this not open PR's for test which have compat for the main repo?

We shouldn't keep separate compats for both

vchuravy commented 4 weeks ago

Yeah I agree that is suboptimal.

vchuravy commented 4 weeks ago

There is an upstream issue for that: https://github.com/JuliaRegistries/CompatHelper.jl/issues/294

vchuravy commented 4 weeks ago

I disabled it for test again