Skip to content

FIX broken sdist#13862

Merged
jnothman merged 1 commit intoscikit-learn:masterfrom
ogrisel:fix-sdist
May 11, 2019
Merged

FIX broken sdist#13862
jnothman merged 1 commit intoscikit-learn:masterfrom
ogrisel:fix-sdist

Conversation

@ogrisel
Copy link
Copy Markdown
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
Copy Markdown
Member Author

ogrisel commented May 10, 2019

Related: #13861.

@jnothman
Copy link
Copy Markdown
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