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

Include license file in source tarball #78

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

CurtLH
Copy link

@CurtLH CurtLH commented Jun 7, 2018

The LICENSE is currently not included in the source tarball.

@CurtLH
Copy link
Author

CurtLH commented Jun 7, 2018

I'm not sure why the test failed, but it doesn't appear to be related to the changes I'm proposing.

Link to error

======================================================================
ERROR: Check that the R notebook generated from Rmd looks the same
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/virtualenv/python3.6.5/lib/python3.6/site-packages/nose/case.py", line 198, in runTest
    self.test(*self.arg)
  File "/home/travis/build/aaren/notedown/tests.py", line 396, in test_R
    knitr = notedown.Knitr()
  File "/home/travis/build/aaren/notedown/notedown/notedown.py", line 600, in __init__
    raise self.KnitrError(message)
notedown.notedown.Knitr.KnitrError: Could not load knitr (needs manual installation).
$ Rscript -e require(knitr)
Loading required package: knitr
Warning message:
In library(package, lib.loc = lib.loc, character.only = TRUE, logical.return = TRUE,  :
  there is no package called ‘knitr’
----------------------------------------------------------------------

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.

1 participant