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

fix: include docs and examples in the sdist tarball #2289

Merged
merged 1 commit into from Mar 19, 2023

Conversation

aucampia
Copy link
Member

Summary of changes

The sdists generated by setuptools included the docs and examples directories, and they are needed for building docs and running tests using the sdist.

This change includes these directories in the sdist tarball.

A test:sdist task is also added to Taskfile.yml which uses the sdists to run pytest and build docs.

Checklist

  • Checked that there aren't other open pull requests for
    the same change.
  • Added tests for any changes that have a runtime impact.
  • Checked that all tests and type checking passes.
  • Considered granting push permissions to the PR branch,
    so maintainers can fix minor issues and keep your PR up to date.

The sdists generated by setuptools included the `docs` and `examples`
directories, and they are needed for building docs and running tests using the
sdist.

This change includes these directories in the sdist tarball.

A `test:sdist` task is also added to `Taskfile.yml` which uses the sdists to run
pytest and build docs.
@aucampia aucampia force-pushed the iwana-20230319T1203-sdist_tests branch from bb6cb01 to 359c5d0 Compare March 19, 2023 14:19
@coveralls
Copy link

Coverage Status

Coverage: 90.782%. Remained the same when pulling 359c5d0 on aucampia:iwana-20230319T1203-sdist_tests into f3d31b0 on RDFLib:main.

@aucampia aucampia marked this pull request as ready for review March 19, 2023 14:39
@aucampia aucampia requested a review from a team March 19, 2023 14:39
@aucampia aucampia added review wanted This indicates that the PR is ready for review ready to merge The PR will be merged soon if no further feedback is provided. labels Mar 19, 2023
@aucampia
Copy link
Member Author

Merging without review, as this has no runtime changes and only includes additional content in sdist.

@aucampia aucampia merged commit 394fb50 into RDFLib:main Mar 19, 2023
23 checks passed
@aucampia aucampia deleted the iwana-20230319T1203-sdist_tests branch April 9, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge The PR will be merged soon if no further feedback is provided. review wanted This indicates that the PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants