samtools / htsjdk

A Java API for high-throughput sequencing data (HTS) formats.
http://samtools.github.io/htsjdk/
283 stars 242 forks source link

remove hardcoded .idx #1568

Closed yash-puligundla closed 3 years ago

yash-puligundla commented 3 years ago

@lbergelson I realized I had hardcoded the ".idx" file format in the exception message after the Pull Request was merged.

Description: When index file is absent, htsjdk errors with "An index is required, but none found" message here

Fix: Error message specifies the expected index file format "An index is required, but none found with file ending " + (index File Format).

codecov-commenter commented 3 years ago

Codecov Report

Merging #1568 (ec1e5c9) into master (d72d73b) will not change coverage. The diff coverage is 50.000%.

@@             Coverage Diff             @@
##              master     #1568   +/-   ##
===========================================
  Coverage     69.841%   69.841%           
  Complexity      9633      9633           
===========================================
  Files            702       702           
  Lines          37611     37611           
  Branches        6108      6108           
===========================================
  Hits           26268     26268           
  Misses          8897      8897           
  Partials        2446      2446           
Impacted Files Coverage Δ
...va/htsjdk/tribble/TribbleIndexedFeatureReader.java 74.346% <50.000%> (ø)