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

WIP: Get things working on v0.5 #212

Merged
merged 5 commits into from
Jul 27, 2016
Merged

Conversation

ranjanan
Copy link
Contributor

@ranjanan ranjanan commented Jul 27, 2016

Added complement and complement!. Fixing tests now. Comments and suggestions welcome as I go along.

Fixes #209

cc: @kmsquire @phaverty @tkelman

@codecov-io
Copy link

codecov-io commented Jul 27, 2016

Current coverage is 92.05% (diff: 100%)

Merging #212 into master will increase coverage by 2.15%

@@             master       #212   diff @@
==========================================
  Files            27         27          
  Lines          2288       2291     +3   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           2057       2109    +52   
+ Misses          231        182    -49   
  Partials          0          0          

Powered by Codecov. Last update ccc1e2f...d24cbed

@@ -2,6 +2,7 @@ using Base.Test
using DataStructures
const IntSet = DataStructures.IntSet
import Compat: String
using Primes
Copy link
Contributor

Choose a reason for hiding this comment

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

Where does this get used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The Primes package provides the primes function, which is deprecated in Base right?

@Keno
Copy link
Contributor

Keno commented Jul 27, 2016

LGTM.

@Keno Keno merged commit 06792b4 into JuliaCollections:master Jul 27, 2016
@ranjanan ranjanan deleted the RA/fix branch July 28, 2016 04:14
@ranjanan
Copy link
Contributor Author

@Keno @StefanKarpinski @tkelman can one of you guys tag a new release please?

@tkelman
Copy link
Contributor

tkelman commented Jul 28, 2016

Sure.

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.

5 participants