search

twhiteaker / CFGeom

CF Convention for Representing Simple Geometry Types
MIT License
9 stars 4 forks source link

Merge R and Python Travis testing scripts #43

Closed bekozi closed 7 years ago

bekozi commented 7 years ago

We are interested in combining the Python and R reference implementations in a single repository. Both implementations using Travis CI for testing. How can we combine the Python and R tests? See #9 for discussion leading to this ticket.

dblodgett-usgs commented 7 years ago

The only way I think this would work well is with docker containers or some super custom scripted approach. Probably not worth it?

dblodgett-usgs commented 7 years ago

https://github.com/travis-ci/travis-ci/issues/4090

bekozi commented 7 years ago

Does look messy doesn't it. The way I thought about it, since we are using a Linux image, is to just install R and needed dependencies during the before_install step. Where are the R package requirements?

bekozi commented 7 years ago

We could also consider using a custom Docker container. We should probably get one together anyway that contains both reference implementations. Maybe this is the first step?

dblodgett-usgs commented 7 years ago

R uses the 'c' language typically. From there, you use the travis-tool script here: https://github.com/dblodgett-usgs/NCDFSG/blob/master/.travis.yml#L17

If there's a way to do most of this within a python job, then you could probably have the R and Python in the same repo? I'm not sure how to do things in the python travis job though...

bekozi commented 7 years ago

I can see this will get a bit complex. Let's hold off for now...but....there must be a way.