Skip to content

Small fixes in album name parsing#96

Merged
snejus merged 3 commits into
mainfrom
dev
Apr 17, 2026
Merged

Small fixes in album name parsing#96
snejus merged 3 commits into
mainfrom
dev

Conversation

@snejus

@snejus snejus commented Apr 17, 2026

Copy link
Copy Markdown
Owner

Fixed

Fixes: #93

snejus added 3 commits April 15, 2026 11:31
Prevent `find_artist` from returning the left part of a split album
name (e.g. "Neon White OST 2") as the artist when it contains a
soundtrack reference (OST, soundtrack, score, etc.).

Also fix a typo: "pinciple" -> "principle" in a docstring.
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 24562018564

Coverage increased (+0.02%) to 91.337%

Details

  • Coverage increased (+0.02%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 5 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

5 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
album_name.py 5 97.06%

Coverage Stats

Coverage Status
Relevant Lines: 1420
Covered Lines: 1295
Line Coverage: 91.2%
Relevant Branches: 196
Covered Branches: 181
Branch Coverage: 92.35%
Branches in Coverage %: Yes
Coverage Strength: 22.76 hits per line

💛 - Coveralls

@snejus snejus merged commit ec4a628 into main Apr 17, 2026
56 checks passed
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.

Beetcamp incorrectly truncates first part of album title and puts it in artist field

2 participants