Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Aqua and JET in tests #130

Merged
merged 3 commits into from
Sep 13, 2022
Merged

Use Aqua and JET in tests #130

merged 3 commits into from
Sep 13, 2022

Conversation

Drvi
Copy link
Collaborator

@Drvi Drvi commented Sep 9, 2022

Using Aqua, this PR automates checking for method ambiguities, unbound type parameters, undefined exports, and compat bounds being set.

Using JET to make sure xparse doesn't have dynamic dispatch or boxed variables in closures. There seem to be code paths leading to dynamic dispatch when parsing Date and Time types.

@codecov
Copy link

codecov bot commented Sep 9, 2022

Codecov Report

Merging #130 (9c692fd) into main (223ec68) will increase coverage by 3.50%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##             main     #130      +/-   ##
==========================================
+ Coverage   86.21%   89.71%   +3.50%     
==========================================
  Files           9        9              
  Lines        2350     2440      +90     
==========================================
+ Hits         2026     2189     +163     
+ Misses        324      251      -73     
Impacted Files Coverage Δ
src/ints.jl 91.17% <50.00%> (+1.47%) ⬆️
src/Parsers.jl 94.51% <100.00%> (+0.24%) ⬆️
src/floats.jl 95.01% <100.00%> (+2.75%) ⬆️
src/strings.jl 96.91% <100.00%> (+0.01%) ⬆️
src/dates.jl 80.24% <0.00%> (+4.70%) ⬆️
src/utils.jl 90.49% <0.00%> (+5.37%) ⬆️
src/ryu.jl 81.20% <0.00%> (+8.36%) ⬆️
... and 1 more

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

@Drvi
Copy link
Collaborator Author

Drvi commented Sep 9, 2022

Hmm, can't reproduce the JET failures locally. Will try on a linux machine. UPDATE: No change :/
UPDATE2: I'm dumb and based my PR on an old main branch... Now the only failing JET-tests are unrelated to this package

Copy link
Member

@quinnj quinnj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome; thanks @Drvi!

@quinnj
Copy link
Member

quinnj commented Sep 13, 2022

The nightly JET failures are being tracked here

@quinnj quinnj closed this Sep 13, 2022
@quinnj quinnj reopened this Sep 13, 2022
@quinnj quinnj merged commit 6297d57 into JuliaData:main Sep 13, 2022
nickrobinson251 added a commit that referenced this pull request Oct 12, 2022
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.

None yet

2 participants