xtensor-stack / xtensor-r

R bindings for xtensor
BSD 3-Clause "New" or "Revised" License
87 stars 15 forks source link

Split xtensor-r and Xtensor.R #82

Closed SylvainCorlay closed 5 years ago

SylvainCorlay commented 5 years ago

@wolfv I leave it to you to merge this!

wolfv commented 5 years ago

Looks good to me!

Is there no way to run the R-only unit test on commit in this repo?

SylvainCorlay commented 5 years ago

Is there no way to run the R-only unit test on commit in this repo?

Not at the moment.

We could install Xtensor.R from GitHub with --configure-args='--novendor' as part of this and check that we can build the same unit test.

wolfv commented 5 years ago

I think that would be cool so that we keep the unit tests on push in here

do you want to merge now or should we figure that out first?

SylvainCorlay commented 5 years ago

Working on the build for Xtensor.R in a separate PR