Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CRAN release soilDB 2.5 #125

Closed
brownag opened this issue Jan 24, 2020 · 4 comments
Closed

CRAN release soilDB 2.5 #125

brownag opened this issue Jan 24, 2020 · 4 comments

Comments

@brownag
Copy link
Member

brownag commented Jan 24, 2020

As near as I can tell, we are ready for soilDB 2.5 to go to CRAN -- as long as everything looks good on the winbuilder side.

Here are some big ticket items we discussed on closed PR #121 -- goal was to have these done for CRAN release.

Done (thanks @smroecker & @dylanbeaudette for your work on these)
#112
#53
#45
#44
#38
#36

Still open:
#1

@brownag
Copy link
Member Author

brownag commented Jan 24, 2020

For the future: see also recent issues and fixes #124 #123 related to examples in soilDB docs

@dylanbeaudette
Copy link
Member

Thanks everyone for addressing these issues. I don't think that we will have consensus on #1 in time for the stats class. I'd like to get all of the usual suspects together and make a decision.

@dylanbeaudette
Copy link
Member

dylanbeaudette commented Jan 27, 2020

Finally figured out why soilDB was falling off CRAN: web APIs are periodically not available due to maintenance and this throws CRAN check errors. A simple fix:

  • added .local_NASIS_defined() to soilDB, returns TRUE | FALSE
  • API-related test have to run, but not continuously on CRAN servers

Be sure to add the following to any test that involves web API access. Tests should be gracefully skipped on machines that don't have a local_nasis ODBC data source defined.

# obvious 
skip_if_offline()
  
# hack for in-house testing only
# WWW services aren't always available and will cause CRAN to drop our package if tests fail
if(! soilDB:::.local_NASIS_defined()) {
   skip("in-house testing only")
}

We may still have problems if examples, even inside of \donttest{}. We will see.

@brownag
Copy link
Member Author

brownag commented Jan 29, 2020

soilDB 2.5.0 is on CRAN as of midday today

https://github.com/ncss-tech/soilDB/releases/tag/2.5

@brownag brownag closed this as completed Jan 29, 2020
@dylanbeaudette dylanbeaudette unpinned this issue Feb 10, 2020
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

No branches or pull requests

2 participants