Skip to content

lyrics: add rest_directory configuration option#6745

Open
hakkilab wants to merge 2 commits into
beetbox:masterfrom
hakkilab:rest-directory-config
Open

lyrics: add rest_directory configuration option#6745
hakkilab wants to merge 2 commits into
beetbox:masterfrom
hakkilab:rest-directory-config

Conversation

@hakkilab

Copy link
Copy Markdown

Description

Fixes #2806

  • Adds a rest_directory configuration option to the lyrics plugin that specifies a directory for ReST output, equivalent to the -r, --write-rest command line argument

  • Documentation.

  • Changelog.

  • Tests.

@hakkilab hakkilab requested a review from a team as a code owner June 15, 2026 10:44
@github-actions github-actions Bot added the lyrics lyrics plugin label Jun 15, 2026
@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.79%. Comparing base (60047df) to head (bdd7a46).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6745      +/-   ##
==========================================
+ Coverage   74.52%   74.79%   +0.26%     
==========================================
  Files         162      162              
  Lines       20818    20818              
  Branches     3295     3295              
==========================================
+ Hits        15515    15571      +56     
+ Misses       4547     4489      -58     
- Partials      756      758       +2     
Files with missing lines Coverage Δ
beetsplug/lyrics.py 90.34% <100.00%> (+10.81%) ⬆️
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hakkilab

Copy link
Copy Markdown
Author

I added another commit to resolve a failing lint check. Please review this PR at your earliest convenience. This is my first contribution so let me know if any changes are needed and I will update the PR. Thanks!
@sampsyo @snejus

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lyrics lyrics plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

lyrics: Configuration option for ReST writing

1 participant