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

restrict constraint on clef lines #921

Merged
merged 1 commit into from
Apr 6, 2022

Conversation

rettinghaus
Copy link
Member

This brings the constraint inline with another one (line 596). As correctly stated there, percussion and tabular clef don't need a line specified, yet <clef shape="perc" /> or <clef shape="TAB" /> were marked as an error.

@github-actions github-actions bot added the Component: Core Schema changes to source/modules/* (assigned automatically) label Feb 24, 2022
@rettinghaus rettinghaus added this to 2022-04-28 ODD Thursday in ODD Meetings Mar 14, 2022
@rettinghaus rettinghaus moved this from 2022-04-28 ODD Thursday to 2022-03-25 ODD Friday in ODD Meetings Mar 21, 2022
@musicEnfanthen musicEnfanthen requested review from musicEnfanthen and removed request for musicEnfanthen March 25, 2022 15:10
Copy link
Member

@musicEnfanthen musicEnfanthen left a comment

Choose a reason for hiding this comment

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

Thank you!

Copy link
Member

@annplaksin annplaksin left a comment

Choose a reason for hiding this comment

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

The solution might not be perfect but is the smallest possible fix.

@musicEnfanthen
Copy link
Member

musicEnfanthen commented Mar 25, 2022

@bwbohl This is ready to be merged. Please feel free to add your second review and merge as time permits.

@musicEnfanthen musicEnfanthen added Status: Ready To Merge indicates that a pull request is ready for merging and removed Status: Needs Review labels Mar 25, 2022
@bwbohl bwbohl self-requested a review March 31, 2022 13:52
@bwbohl
Copy link
Member

bwbohl commented Apr 6, 2022

thx for your work @rettinghaus and thanks for the review @musicEnfanthen

@bwbohl bwbohl merged commit eaf70f9 into music-encoding:develop Apr 6, 2022
@rettinghaus rettinghaus deleted the develop-clef branch April 7, 2022 18:03
@musicEnfanthen musicEnfanthen removed the Status: Ready To Merge indicates that a pull request is ready for merging label Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Core Schema changes to source/modules/* (assigned automatically)
Projects
No open projects
ODD Meetings
  
2022-03-25 ODD Friday
Development

Successfully merging this pull request may close these issues.

None yet

4 participants