Skip to content

Conversation

ranjanan
Copy link
Contributor

Add REQUIRE file to tests for StatsBase, because hist is deprecated, and sub is now view

@coveralls
Copy link

coveralls commented Jul 28, 2016

Coverage Status

Coverage increased (+0.02%) to 97.924% when pulling f186dde on ranjanan:RA/depwarn into 3e666ec on JuliaLang:master.

module SortingAlgorithms

using Compat
import Compat.view
Copy link
Contributor

@tkelman tkelman Jul 29, 2016

Choose a reason for hiding this comment

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

put a minimum version on Compat in REQUIRE

@coveralls
Copy link

coveralls commented Aug 1, 2016

Coverage Status

Coverage increased (+0.02%) to 97.924% when pulling db33672 on ranjanan:RA/depwarn into 3e666ec on JuliaLang:master.

@ranjanan
Copy link
Contributor Author

ranjanan commented Aug 2, 2016

The reason this is failing is because StatsBase does not support v0.3. I used StatsBase because the hist function is deprecated in Base. Would it be okay to drop v0.3 support for this package?

@tkelman
Copy link
Contributor

tkelman commented Aug 2, 2016

I think dropping 0.3 support here is fine.

@ranjanan ranjanan changed the title Fix deprecation warnings Fix deprecation warnings, remove v0.3 support Aug 2, 2016
@coveralls
Copy link

coveralls commented Aug 2, 2016

Coverage Status

Coverage increased (+0.02%) to 97.924% when pulling 3688f32 on ranjanan:RA/depwarn into 3e666ec on JuliaLang:master.

@kmsquire kmsquire merged commit daf9126 into JuliaCollections:master Aug 2, 2016
@ranjanan ranjanan deleted the RA/depwarn branch August 3, 2016 05:51
@ranjanan
Copy link
Contributor Author

ranjanan commented Aug 3, 2016

@kmsquire @tkelman @ViralBShah Could one of you guys tag a new version? Thanks.

@tkelman
Copy link
Contributor

tkelman commented Aug 3, 2016

(will have to be a new minor version via Pkg.tag("SortingAlgorithms", :minor) since 0.3 support was dropped)

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.

4 participants