Skip to content

Conversation

timholy
Copy link
Member

@timholy timholy commented Feb 25, 2023

Keyword dispatch methods went away in Julia 1.9, replaced by the
generic kwcall. In #102 I disabled the corresponding test.
That was misguided because we still want to be able to recover the
source definition, regardless of the details of the underlying
implementation.

Keyword dispatch methods went away in Julia 1.9, replaced by the
generic `kwcall`. In #102 I disabled the corresponding test.
That was misguided because we still want to be able to recover the
source definition, regardless of the details of the underlying
implementation.
@codecov
Copy link

codecov bot commented Feb 25, 2023

Codecov Report

Base: 95.25% // Head: 91.39% // Decreases project coverage by -3.87% ⚠️

Coverage data is based on head (f244331) compared to base (5aa2d96).
Patch coverage: 37.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #105      +/-   ##
==========================================
- Coverage   95.25%   91.39%   -3.87%     
==========================================
  Files           3        3              
  Lines         232      244      +12     
==========================================
+ Hits          221      223       +2     
- Misses         11       21      +10     
Impacted Files Coverage Δ
src/CodeTracking.jl 86.61% <37.50%> (-6.46%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@timholy
Copy link
Member Author

timholy commented Feb 25, 2023

I don't understand why it's reporting no coverage for this code; it clearly is covered locally.

@timholy timholy merged commit 63bbeaa into master Feb 25, 2023
@timholy timholy deleted the teh/kwcall branch February 25, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant