Skip to content

Conversation

@staticfloat
Copy link
Contributor

Slightly simplify a few things as we can now directly specify what version of a dependency we support.

This is, naturally, 1.3+ only, I'm using it to try and make the GtkArtifactTest repo 100% JLL package based.

Slightly simplify a few things as we can now directly specify what
version of a dependency we support.
Copy link
Member

@timholy timholy left a comment

Choose a reason for hiding this comment

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

Love it!

Co-Authored-By: Tim Holy <tim.holy@gmail.com>
@codecov
Copy link

codecov bot commented Nov 29, 2019

Codecov Report

Merging #168 into master will decrease coverage by 2.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   71.12%   69.11%   -2.02%     
==========================================
  Files           2        2              
  Lines         329      327       -2     
==========================================
- Hits          234      226       -8     
- Misses         95      101       +6
Impacted Files Coverage Δ
src/libmagickwand.jl 65.15% <ø> (-0.18%) ⬇️
src/ImageMagick.jl 75.19% <100%> (-4.81%) ⬇️

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 bc8a0e4...835c2b0. Read the comment docs.

@timholy
Copy link
Member

timholy commented Nov 29, 2019

We're failing on nightly on a couple of platforms, but since it works on 1.3 I say let's merge this!

I might wait a day or so and see if we can fix nightly before registering a new version. Any reason not to call it 1.0?

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