ChainRulesCore.jl
ChainRulesCore.jl copied to clipboard
WIP: add whitespace check
Codecov Report
Merging #463 (9fd609c) into master (bdb52bb) will not change coverage. The diff coverage is
n/a.
:exclamation: Current head 9fd609c differs from pull request most recent head 066711e. Consider uploading reports for the commit 066711e to get more accurate results
@@ Coverage Diff @@
## master #463 +/- ##
=======================================
Coverage 92.98% 92.98%
=======================================
Files 14 14
Lines 812 812
=======================================
Hits 755 755
Misses 57 57
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update bdb52bb...066711e. Read the comment docs.
Rather than a check which can fail, can we use the reviewdog action to insert a review suggestion for how to fix? That way do not have to change from the screen where it fails back the my editor.
I'll look into that, once I figured out the other issues.
@simeonschaub: in case you hadn't spotted it in the related issue yet, https://github.com/JuliaDiff/ChainRulesCore.jl/issues/462#issuecomment-925593793