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

updates for jsonlite. #32

Merged
merged 5 commits into from Jan 27, 2017

Conversation

@restonslacker
Copy link
Contributor

@restonslacker restonslacker commented Jan 27, 2017

#31 needed to switch to '[[' from '[' for file upload request. all tests in simpleTests.R pass with this change

@restonslacker
Copy link
Contributor Author

@restonslacker restonslacker commented Jan 27, 2017

not quite sure what the issue with travis is. maybe because the tests are designed to throw errors?

.travis.yml Outdated
@@ -17,7 +17,7 @@ before_install:
- ./travis-tool.sh bootstrap

install:
- ./travis-tool.sh install_r RJSONIO
- ./travis-tool.sh install_r jsonlite
- ./travis-tool.sh install_aptget r-cran-curl

This comment has been minimized.

@eddelbuettel

eddelbuettel Jan 27, 2017
Owner

Sorry, that again. Try adding `r-cran-jsonlite' on the second line, remove the first. Will also make it faster (no compiles)

This comment has been minimized.

@restonslacker

restonslacker Jan 27, 2017
Author Contributor

arg! sorry i copied the wrong bit...

@eddelbuettel eddelbuettel merged commit a1feee3 into eddelbuettel:master Jan 27, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.