beetcamp
beetcamp copied to clipboard
Fix artist parsing for tracks with album/series titles in names
- [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] Reduced complexity in
- [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.
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 | |
|---|---|
| Change from base Build 17655664207: | 0.2% |
| Covered Lines: | 1256 |
| Relevant Lines: | 1388 |
💛 - Coveralls
@copilot can you fix the linting issues please?
@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