Skip to content

Conversation

@capnrefsmmat
Copy link
Contributor

.defaultStringsAsFactors is a list with one named element,
$stringsAsFactors, so we pass the entire list to options() to set that
option back to its default.

My fault when I introduced this trick in #290. It was causing this error message after tests completed:

Error in options(stringsAsFactors = .defaultStringsAsFactors) : 
  invalid value for 'stringsAsFactors'

.defaultStringsAsFactors is a list with one named element,
$stringsAsFactors, so we pass the entire list to options() to set that
option back to its default.
@capnrefsmmat capnrefsmmat merged commit 1ef8ff1 into r-pkg-devel Nov 24, 2020
@taylor-arnold taylor-arnold deleted the r/fix-tests branch March 19, 2021 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants