Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add clarification to matching option items #160

Merged
merged 1 commit into from Mar 15, 2022

Conversation

rdswift
Copy link
Collaborator

@rdswift rdswift commented Mar 6, 2022

Summary

This is a…

  • Correction
  • Addition
  • Restructuring
  • Minor / simple change (like a typo)
  • Other

Reason for the Change

Some confusion regarding use of the matching options. See discussion at https://community.metabrainz.org/t/difference-between-matching-file-lookups-and-matching-files-to-tracks/575402

Description of the Change

Add clarification to the descriptions for the settings, as provided in the response by outsidecontext.

Additional Action Required

None.

@rdswift rdswift added the documentation Improvements or additions to documentation label Mar 6, 2022
@rdswift rdswift requested a review from phw March 6, 2022 16:02
@rdswift rdswift self-assigned this Mar 6, 2022
Copy link
Member

@phw phw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that clarifies these settings a lot. Thanks for this update

@rdswift rdswift merged commit b9e308d into metabrainz:master Mar 15, 2022
@rdswift rdswift deleted the matching_clarification branch March 15, 2022 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
2 participants