Skip to content
This repository has been archived by the owner on Sep 1, 2020. It is now read-only.

Fix eltype for chained generators #97

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix eltype for chained generators #97

wants to merge 1 commit into from

Conversation

iamed2
Copy link
Collaborator

@iamed2 iamed2 commented Jan 23, 2017

Fixes #90

Still need to add a test

@codecov-io
Copy link

codecov-io commented Jan 23, 2017

Codecov Report

Merging #97 into master will increase coverage by 0.29%.

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   94.11%   94.41%   +0.29%     
==========================================
  Files           1        1              
  Lines         374      376       +2     
==========================================
+ Hits          352      355       +3     
+ Misses         22       21       -1
Impacted Files Coverage Δ
src/Iterators.jl 94.41% <50%> (+0.29%)

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 b332f56...ad5a926. Read the comment docs.

@tkelman tkelman closed this Feb 2, 2017
@tkelman tkelman reopened this Feb 2, 2017
@iamed2
Copy link
Collaborator Author

iamed2 commented Feb 2, 2017

If we want to continue to support 0.4 I'll make an additional test file for 0.5+ tests that use generator syntax.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants