Skip to content

Commit

Permalink
cran comment, shorter
Browse files Browse the repository at this point in the history
  • Loading branch information
khufkens committed May 22, 2023
1 parent f80cef8 commit 8613ccb
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions cran-comments.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,14 +22,9 @@ a breakdown of coverage statistics I refer to:

https://app.codecov.io/gh/bluegreen-labs/appeears

Limited functions are documented with "dynamic" examples which are run.
The code required to trap potential errors if the service is down
exceeds that of actual function coverage (which can be confusing, and the
reason I tend to avoid this). However, I will retain these changes
as per review request. Functions covered in this way are, rs_layers() and
rs_products() which require no authentication. rs_build_task() is now exposed
as well (not wrapped in dontrun). All other function touch upon some form of
authentication (and can't be tested due to credentials).
Limited functions are documented with "dynamic" examples which are run (on CRAN).
Functions covered in this way are, rs_layers() and rs_products() which require no authentication. rs_build_task() is now exposed as well (not wrapped in dontrun).
All other function touch upon some form of authentication (and can't be tested due to credentials).

Furthermore, I added Elio Campitelli as contributor, as the R6 refactoring of
{ecmwfr} was mostly his work. I've borrowed this for the package in hindsight
Expand Down

0 comments on commit 8613ccb

Please sign in to comment.