Skip to content

Conversation

@tomhughes
Copy link
Contributor

The "list" stream is a Transform stream which is writable at
least in current node versions.

The "list" stream is a Transform stream which is writable at
least in current node versions.
@coveralls
Copy link

coveralls commented Dec 23, 2016

Coverage Status

Coverage remained the same at 97.685% when pulling 357ae1b on tomhughes:invalid-writable into a66c37d on mapbox:master.

@coveralls
Copy link

coveralls commented Dec 23, 2016

Coverage Status

Coverage remained the same at 97.685% when pulling 357ae1b on tomhughes:invalid-writable into a66c37d on mapbox:master.

@rclark
Copy link
Contributor

rclark commented Jan 2, 2017

Tests are failing on CI due to an unrelated issue. Diggin'

@tomhughes
Copy link
Contributor Author

I guess we should add node 6/7 to the CI checks as well? It was 6.9.2 where I was seeing this test fail...

@tomhughes
Copy link
Contributor Author

Oh that roundtrip size check often fails. I relax the tolerance on that when building for Fedora:

http://pkgs.fedoraproject.org/cgit/rpms/nodejs-tilelive.git/tree/nodejs-tilelive-roundtrip.patch

It tends to vary by platform exactly how much tolerance is needed.

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.

3 participants