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

Fix typealias depwarns on 0.6 #25

Merged
merged 1 commit into from
Feb 18, 2017
Merged

Fix typealias depwarns on 0.6 #25

merged 1 commit into from
Feb 18, 2017

Conversation

timholy
Copy link
Member

@timholy timholy commented Feb 18, 2017

Broken on 0.6 due to JuliaLang/julia#20658, but meanwhile we can fix a bunch of depwarns.

@codecov-io
Copy link

codecov-io commented Feb 18, 2017

Codecov Report

Merging #25 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #25   +/-   ##
======================================
  Coverage    86.5%   86.5%           
======================================
  Files           9       9           
  Lines        1089    1089           
======================================
  Hits          942     942           
  Misses        147     147
Impacted Files Coverage Δ
src/ImageFiltering.jl 100% <ø> (ø)

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 06624f0...b1d4df2. Read the comment docs.

- typealias, abstract, and inner constructor depwarns
- change in NTuple{N} specificity
- deprecation of .+ as separate functions for broadcasting

Also runs ambiguity tests only on 0.5
@timholy timholy force-pushed the teh/typealias_depwarns branch from afca453 to b1d4df2 Compare February 18, 2017 14:52
@timholy timholy merged commit a643d7f into master Feb 18, 2017
@timholy timholy deleted the teh/typealias_depwarns branch February 18, 2017 15:16
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