Skip to content

migrate to new python packaging #799

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

migrate to new python packaging #799

wants to merge 14 commits into from

Conversation

@reece
Copy link
Member

@reece reece commented Sep 8, 2025 •
edited
Loading

This PR migrates hgvs to the new python packaging.

Note: In order to get tests to pass, I also had to:

  • mark babelfish as an extra (exclude from default testing), which is appropriate for hgvs.extras.* modules anyway
  • unset HGVS_CACHE_MODE in Makefile due to an(other) obscure bug in caching behavior
  • make vcrpy record mode "none" by default

To test UX, please:

  • clone
  • Read the README.md. Note that uv must be installed globally is implemented right now.
  • rm -fr venv .venv
  • make devready
  • make test

@reece reece linked an issue Sep 8, 2025 that may be closed by this pull request
Update / modernize packaging #691
Open
@github-actions
Copy link

github-actions bot commented Sep 16, 2025

This issue is stale because it has been open 180 days with no activity.
Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale Issue is stale and subject to automatic closing label Sep 16, 2025
@reece reece removed the stale Issue is stale and subject to automatic closing label Oct 14, 2025
@reece reece mentioned this pull request Oct 14, 2025
Open
@github-actions
Copy link

github-actions bot commented Oct 21, 2025

This issue is stale because it has been open 180 days with no activity.
Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale Issue is stale and subject to automatic closing label Oct 21, 2025
@github-actions github-actions bot removed the stale Issue is stale and subject to automatic closing label Nov 1, 2025
@codecov
Copy link

codecov bot commented Nov 2, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@reece reece marked this pull request as ready for review November 2, 2025 20:40
@reece reece requested a review from a team as a code owner November 2, 2025 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Update / modernize packaging

2 participants

@reece