Skip to content

Ensure concatenated reference FASTA temp file has .fasta suffix#281

Merged
wwood merged 2 commits intoxmapperfrom
codex/fix-bam-file-handling-for-.fasta-extension
Nov 12, 2025
Merged

Ensure concatenated reference FASTA temp file has .fasta suffix#281
wwood merged 2 commits intoxmapperfrom
codex/fix-bam-file-handling-for-.fasta-extension

Conversation

@wwood
Copy link
Owner

@wwood wwood commented Nov 11, 2025

Summary

  • add a .fasta suffix when generating concatenated reference FASTA files so mappers that require the extension accept the temp file
  • add a unit test to guard the concatenated FASTA extension and content
  • clean up Clippy warnings in existing code that were triggered by the new lint run

Testing

  • cargo clippy -- -D warnings
  • cargo test mapping_index_maintenance::tests::concatenated_fasta_file_has_required_extension

Codex Task

@wwood wwood merged commit 2b82bcd into xmapper Nov 12, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments