Skip to content

fix: address Copilot review, bump to v0.44.3 - #93

Merged
berntpopp merged 1 commit into
mainfrom
fix/92-copilot-feedback
Apr 7, 2026
Merged

fix: address Copilot review, bump to v0.44.3#93
berntpopp merged 1 commit into
mainfrom
fix/92-copilot-feedback

Conversation

@berntpopp

Copy link
Copy Markdown
Owner

Summary

  • Compute primer overhead dynamically in boundary tests instead of hardcoding vntr_bp values
  • Clarify test docstrings: tests operate on raw bp sizes, not VNTR repeat counts
  • Bump version to 0.44.3

Addresses Copilot review comments on #92.

Test plan

  • All 14 amplicon extractor tests pass
  • ruff check clean

🤖 Generated with Claude Code

- Compute primer overhead dynamically instead of hardcoding vntr_bp
- Clarify docstrings: tests use raw bp sizes, not repeat counts
- Bump version to 0.44.3

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@berntpopp
berntpopp merged commit ebe8c00 into main Apr 7, 2026
5 checks passed
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@berntpopp
berntpopp deleted the fix/92-copilot-feedback branch April 7, 2026 13:04
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