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 in manifest.in #666

Merged
merged 1 commit into from Sep 5, 2017

Conversation

Projects
None yet
2 participants
@Carreau
Copy link
Contributor

Carreau commented Sep 5, 2017

@mpacer, you are the release manager of nbconvert , do you want to make a patch release with just this ? We can have further discussion about other issues with manifest.in.

@Carreau Carreau requested a review from mpacer Sep 5, 2017

@mpacer

This comment has been minimized.

Copy link
Member

mpacer commented Sep 5, 2017

I can do the patch release with this and the bad fix to #663.

@mpacer mpacer merged commit d5bb6de into jupyter:master Sep 5, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@mpacer

This comment has been minimized.

Copy link
Member

mpacer commented Sep 5, 2017

eh i won't include the fix to #663 in this just for the sake of ease

@mpacer mpacer added this to the 5.3.1 milestone Sep 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment