Skip to content
This repository has been archived by the owner on May 5, 2019. It is now read-only.

Add upper limit on DataStreams and WeakRefStrings version #87

Merged
merged 1 commit into from Aug 28, 2017

Conversation

nalimilan
Copy link
Member

@nalimilan nalimilan commented Aug 25, 2017

Next major releases of DataStreams and WeakRefStrings will move from Nullable to Union{T, Null} and will not work with the package in its current state.

Cf. JuliaLang/METADATA.jl#10915.

@nalimilan nalimilan requested a review from quinnj August 25, 2017 17:53
@cjprybol
Copy link
Contributor

cjprybol commented Aug 25, 2017

@quinnj does this need an upper limit to WeakRefStrings too? Looks like the latest release v0.3.0 of WRS includes the new changes for Nulls and DataTables is targeting the format used in v0.2.0 yes, JuliaLang/METADATA.jl#10915 (comment)

@quinnj
Copy link
Member

quinnj commented Aug 25, 2017

The DataStreams upper bound should already include the WRS bound, so not technically, but couldn't hurt.

Next major releases of DataStreams and WeakRefStrings will move from
`Nullable` to `Union{T, Null}` and will not work with the package
in its current state.
@nalimilan
Copy link
Member Author

I've also added the upper bound for WeakRefStrings, can't hurt.

@nalimilan nalimilan changed the title Add upper limit on DataStreams version Add upper limit on DataStreams and WeakRefStrings version Aug 25, 2017
@cjprybol
Copy link
Contributor

Failing on nightlies due to an issue in CSV https://ci.appveyor.com/project/nalimilan/datatables-jl/build/1.0.379/job/ddcpq7vwdvg5dwdd#L4871. I've tagged a new release which should include the fix JuliaLang/METADATA.jl#10923

@nalimilan nalimilan merged commit 5199293 into master Aug 28, 2017
@nalimilan nalimilan deleted the nl/datastreams branch August 28, 2017 12:11
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.

None yet

3 participants