Bump up R version requirement about ecmwfr HOT 6 CLOSED

Fixed with Travis CI integration.

from ecmwfr.

I don't know. I wouldn't want to increase the requirements unless we are using some functionality that is not available in previous versions and hard to backport. I am not aware of any used in this package.

I use R 3.6 in all of my machines and in the last workshop I gave I used RStudio Cloud, which I think also has the most up-to-date version of R. So I don't have much experience working with old versions. What has been your experience? Does ecmwfr fail often with R < 3.6?

from ecmwfr.

You are probably right, problem is that I don't test anymore on 3.4. I used to have at least one system running on 3.4 but I've upgraded all of them to >3.4. Otherwise the issue with wf_request() would have been caught earlier. I'll include a new line in the travis file.

from ecmwfr.

For what is worth,we might get away with a lower version if we wanted to.

https://ateucher.shinyapps.io/check_r_versions_of_package_dependencies/

Is 3.3.0 available to test in Travis?

from ecmwfr.

Now I'm looking at the issue backwards and trying to figure out if it's possible to bump down the R version required. 🤣

Looking at the dependencies, why is {stars} suggested? I don't see any stars:: or library(stars) anywhere in the code. Is it a vestigial dependency?

from ecmwfr.

from ecmwfr.