beetcamp icon indicating copy to clipboard operation
beetcamp copied to clipboard

Fix artist parsing for tracks with album/series titles in names

Open Copilot opened this issue 5 months ago • 3 comments

  • [x] Fix artist parsing for tracks with album/series titles in names
  • [x] Add comprehensive test coverage for the carrie z issue
  • [x] Fix linting issues:
    • [x] Reduced complexity in fix_track_artists() method by extracting helper methods
    • [x] Fixed line length violations by breaking long lines
    • [x] Removed trailing whitespace
    • [x] Fixed blank lines with whitespace
  • [x] All tests passing including the new test case
  • [x] All linting checks (flake8, pylint) passing

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot avatar Sep 13 '25 19:09 Copilot

Pull Request Test Coverage Report for Build 17705539809

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 88.066%

Totals Coverage Status
Change from base Build 17655664207: 0.2%
Covered Lines: 1256
Relevant Lines: 1388

💛 - Coveralls

coveralls avatar Sep 13 '25 19:09 coveralls

@copilot can you fix the linting issues please?

snejus avatar Sep 14 '25 02:09 snejus

@copilot can you fix the linting issues please?

Fixed all linting issues. The changes include:

  • Reduced complexity in fix_track_artists() method by extracting helper methods (_should_fix_album_title_artists(), _fix_album_title_artists(), _fix_the_artist(), _fix_problematic_artists(), _split_remaining_titles())
  • Fixed line length violations by breaking long lines
  • Removed trailing whitespace and blank lines with whitespace
  • All flake8 and pylint checks now pass

Commit: 9b4c72a

Copilot avatar Sep 14 '25 03:09 Copilot