Skip to content

Fix text install#515

Merged
rwood-97 merged 8 commits intomainfrom
fix_text_install
Oct 30, 2024
Merged

Fix text install#515
rwood-97 merged 8 commits intomainfrom
fix_text_install

Conversation

@rwood-97
Copy link
Copy Markdown
Collaborator

@rwood-97 rwood-97 commented Oct 30, 2024

Summary

This PR fixes an issue with pushign 1.5.0 to PyPI due to direct dependencies in the text requirements. See https://peps.python.org/pep-0440/#direct-references.

Instead of having these as extras I have added a text-requirements.txt file which should be a work around for getting this to PyPI

This will need to 1.5.1

Describe your changes

Describe changes/updates you have made to the code.

Checklist before assigning a reviewer (update as needed)

  • Self-review code
  • Ensure submission passes current tests
  • Add tests
  • Update relevant docs
  • Update changelog

Reviewer checklist

Please add anything you want reviewers to specifically focus/comment on.

  • Everything looks ok?

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 74.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
setup.py 0.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rwood-97 rwood-97 merged commit 84e0709 into main Oct 30, 2024
@rwood-97 rwood-97 deleted the fix_text_install branch October 30, 2024 15:17
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.

2 participants