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

Define missing tokens. #110

Merged
merged 1 commit into from
Sep 22, 2015
Merged

Define missing tokens. #110

merged 1 commit into from
Sep 22, 2015

Conversation

Carreau
Copy link
Member

@Carreau Carreau commented Sep 21, 2015

Need a test.

Closes jupyter/notebook#466

(and maybe a few other PRs)

Need a test.

Closes jupyter/notebook#466

(and maybe a few other PRs)
@Carreau Carreau added this to the 4.1 milestone Sep 21, 2015
@minrk
Copy link
Member

minrk commented Sep 22, 2015

@Carreau I'm going to assume that since you made another Issue for the tests, this should be merged now. Merging away...

minrk added a commit that referenced this pull request Sep 22, 2015
@minrk minrk merged commit 306395d into jupyter:master Sep 22, 2015
@mpacer
Copy link
Member

mpacer commented Sep 22, 2015

I don't actually know how this passed — when I run nosetests locally on using ipython I get three errors running $ nosetests nbconvert/tests/test_nbconvertapp.py. Is the tests directory intentionally not included in the travis build?

@Carreau
Copy link
Member Author

Carreau commented Sep 22, 2015

Do you want to make a PR in nbconvert/tests/test_nbconvertapp.py that make the test fails to see if travis fail ? Cause the test should be included in travis testing.

There might be a skipif when tex not included.

@Carreau Carreau deleted the fix-token branch September 22, 2015 23:15
@Carreau
Copy link
Member Author

Carreau commented Sep 22, 2015

I get 185 test locally 1 fail.

@Carreau
Copy link
Member Author

Carreau commented Sep 22, 2015

Reverting something wrong with my system, this is not actually needed, and travis does not test PDF.

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.

None yet

3 participants