Skip to content

Split pybigtools lib.rs into submodules#91

Merged
jackh726 merged 12 commits intojackh726:masterfrom
nvictus:refactor-pybt-lib
Oct 21, 2025
Merged

Split pybigtools lib.rs into submodules#91
jackh726 merged 12 commits intojackh726:masterfrom
nvictus:refactor-pybt-lib

Conversation

@nvictus
Copy link
Collaborator

@nvictus nvictus commented Oct 16, 2025

The pybigtools bindings are a bit unwieldy as a single rust module. I split the reader and writer pyo3 structs, errors, and querying utilities into separate modules to improve maintainability.

@nvictus nvictus requested a review from jackh726 October 16, 2025 18:11
@jackh726
Copy link
Owner

Good idea, but looks like something got shuffled in the merge :)

@nvictus
Copy link
Collaborator Author

nvictus commented Oct 17, 2025

Not sure what happened. Did it over again. Looks like it works now. I also added developer instructions in the pybigtools readme and updated the python linting and testing workflows in CI.

@jackh726 jackh726 merged commit 3844b58 into jackh726:master Oct 21, 2025
6 checks passed
@jackh726
Copy link
Owner

Thanks @nvictus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants