Comonicon.jl icon indicating copy to clipboard operation
Comonicon.jl copied to clipboard

Fix printing of Vararg on Julia 1.7

Open jakobnissen opened this issue 2 years ago • 2 comments

Recent versions of Julia has moved Vararg from being a Type to being its own thing. This means it will error if T is a vararg and you try to do isa to <: operations with it. This caused an error, which is fixed here by explicitly testing if T is a Vararg.

Fixes #261

Note that THIS IS NOT TESTED! I couldn't figure out how to write a test for this.

jakobnissen avatar Oct 25 '23 06:10 jakobnissen

could you just copy-paste the example from that issue in the test? that should be sufficient as a test

Roger-luo avatar Oct 25 '23 17:10 Roger-luo

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Files Coverage Δ
src/ast/printing.jl 84.69% <80.00%> (+84.69%) :arrow_up:

... and 22 files with indirect coverage changes

:loudspeaker: Thoughts on this report? Let us know!.

codecov[bot] avatar Oct 25 '23 17:10 codecov[bot]