Skip to content

Images v0.25.1#994

Merged
johnnychen94 merged 1 commit intoJuliaImages:masterfrom
IanButterworth:patch-1
Jan 5, 2022
Merged

Images v0.25.1#994
johnnychen94 merged 1 commit intoJuliaImages:masterfrom
IanButterworth:patch-1

Conversation

@IanButterworth
Copy link
Copy Markdown
Member

@IanButterworth IanButterworth commented Jan 4, 2022

Given #993 has been merged it would be good to get a new patch release to free up the ecosystem for the new ImageIO version.

As an aside, I think we've tagged a couple of non-breaking releases as breaking releases for ImageIO.
I think the rationale we've had has been that it makes sense when adding new backends, but I think it's not strictly a breaking change and causes downstream noise (like this)

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 4, 2022

Codecov Report

Merging #994 (51b500d) into master (4e2d1b8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #994   +/-   ##
=======================================
  Coverage   87.72%   87.72%           
=======================================
  Files           8        8           
  Lines         823      823           
=======================================
  Hits          722      722           
  Misses        101      101           

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 4e2d1b8...51b500d. Read the comment docs.

@johnnychen94 johnnychen94 merged commit e9a3745 into JuliaImages:master Jan 5, 2022
@IanButterworth IanButterworth deleted the patch-1 branch January 5, 2022 02:09
@johnnychen94
Copy link
Copy Markdown
Member

johnnychen94 commented Jan 5, 2022

As an aside, I think we've tagged a couple of non-breaking releases as breaking releases for ImageIO.
I think the rationale we've had has been that it makes sense when adding new backends, but I think it's not strictly a breaking change and causes downstream noise (like this)

Because the new version drops the old LTS 1.0 Julia compatibility, I feel it's a good reason to bump a minor release.

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.

2 participants