nextstrain / augur

Pipeline components for real-time phylodynamic analysis
https://docs.nextstrain.org/projects/augur/
GNU Affero General Public License v3.0
268 stars 128 forks source link

chore(ancestral): more informative AugurError message #1662

Closed corneliusroemer closed 2 weeks ago

corneliusroemer commented 2 weeks ago

Description of proposed changes

Related issue(s)

Checklist

codecov[bot] commented 2 weeks ago

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 72.27%. Comparing base (efabf11) to head (c5e74b3). Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
augur/ancestral.py 33.33% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1662 +/- ## ======================================= Coverage 72.26% 72.27% ======================================= Files 79 79 Lines 8271 8273 +2 Branches 1691 1691 ======================================= + Hits 5977 5979 +2 Misses 2009 2009 Partials 285 285 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

genehack commented 2 weeks ago

Unsure if updating an error message constitutes a "functional change" that would require a Changelog entry…

corneliusroemer commented 2 weeks ago

Thanks! It's such a small change, I won't pollute the changelog with it.