Skip to content

install testthat from GitHub - #389

Merged
hadley merged 3 commits into
r-lib:masterfrom
krlmlr:test2
Dec 3, 2013
Merged

install testthat from GitHub#389
hadley merged 3 commits into
r-lib:masterfrom
krlmlr:test2

Conversation

@krlmlr

@krlmlr krlmlr commented Dec 2, 2013

Copy link
Copy Markdown
Member

so that tests are actually run on Travis (they were simply skipped before)

Comment thread .travis.yml Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you want to do this?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cargo cult. We may as well leave it out if you prefer.

@krlmlr

krlmlr commented Dec 2, 2013

Copy link
Copy Markdown
Member Author

Travis shows errors because texi2dvi (i.e., LaTeX) is not installed. Do we want to install LaTeX for four tests, or do we just skip them if texi2dvi is not available?

@krlmlr

krlmlr commented Dec 2, 2013

Copy link
Copy Markdown
Member Author

Ready for merging. Added LaTeX for now so that the current codebase doesn't throw errors.

hadley added a commit that referenced this pull request Dec 3, 2013
Refinements to .travis.yml
@hadley
hadley merged commit 92e103c into r-lib:master Dec 3, 2013
@krlmlr
krlmlr deleted the test2 branch December 4, 2013 23:26
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

Successfully merging this pull request may close these issues.

2 participants