Skip to content

FIX broken sdist #13862

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

Merged
merged 1 commit into from
May 11, 2019
Merged

FIX broken sdist #13862

merged 1 commit into from
May 11, 2019

Conversation

ogrisel
Copy link
Member

@ogrisel ogrisel commented May 10, 2019

  • missing *.tp files (cython templates) from MANIFST.in
  • missing __init__.py file to turn a tests folder into a package to
    be shipped with the tarball

Fix #13859.

Note: we probably need to improve our CI to also test for the installation of the generated tarball.

- missing *.tp files (cython templates) from MANIFST.in
- missing __init__.py file to turn a tests folder into a package to
  be shipped with the tarball
@ogrisel
Copy link
Member Author

ogrisel commented May 10, 2019

Related: #13861.

@jnothman
Copy link
Member

If not our ci then our release process/docs. Sorry for not checking.

@jnothman jnothman merged commit eb1f5f2 into scikit-learn:master May 11, 2019
@ogrisel ogrisel deleted the fix-sdist branch May 11, 2019 10:56
jnothman pushed a commit to jnothman/scikit-learn that referenced this pull request May 14, 2019
- missing *.tp files (cython templates) from MANIFST.in
- missing __init__.py file to turn a tests folder into a package to
  be shipped with the tarball
jnothman pushed a commit to jnothman/scikit-learn that referenced this pull request May 14, 2019
- missing *.tp files (cython templates) from MANIFST.in
- missing __init__.py file to turn a tests folder into a package to
  be shipped with the tarball
koenvandevelde pushed a commit to koenvandevelde/scikit-learn that referenced this pull request Jul 12, 2019
- missing *.tp files (cython templates) from MANIFST.in
- missing __init__.py file to turn a tests folder into a package to
  be shipped with the tarball
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0.21.0 sdist appears to be invalid
3 participants