Skip to content

Conversation

@KristofferC
Copy link
Member

@KristofferC KristofferC commented Feb 17, 2020

Add a test for JuliaLang/julia#34680. This should (hopefully) fail on Windows and pass when JuliaLang/julia#34754 is merged.

@codecov-io
Copy link

codecov-io commented Feb 17, 2020

Codecov Report

Merging #341 into master will decrease coverage by 4.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #341      +/-   ##
==========================================
- Coverage   83.98%   79.93%   -4.05%     
==========================================
  Files           2        2              
  Lines         331      324       -7     
==========================================
- Hits          278      259      -19     
- Misses         53       65      +12
Impacted Files Coverage Δ
src/juliaconfig.jl 80.64% <0%> (-9.68%) ⬇️
src/PackageCompiler.jl 79.86% <0%> (-3.47%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a8bdc6...5bb6790. Read the comment docs.

@DilumAluthge
Copy link
Member

Needs a rebase?

@KristofferC KristofferC force-pushed the kc/absolute_path_ccall branch 2 times, most recently from 46a1889 to 90e57b9 Compare February 18, 2020 10:35
@KristofferC KristofferC force-pushed the kc/absolute_path_ccall branch from 90e57b9 to 5bb6790 Compare March 5, 2020 09:51
@DilumAluthge
Copy link
Member

@KristofferC Is this still needed?

I took a look at the merge conflicts, but I'm not familiar enough with this PR to be able to resolve the merge conflicts quickly.

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.

3 participants