-
Notifications
You must be signed in to change notification settings - Fork 64
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
Fix/934 #992
Fix/934 #992
Conversation
As `<exhibHist>`, `<treatHist>` and `<treatSched>` have been remodelled after `<provenance>` in music-encoding#911, the corresponding Schematron rule to report an error in work and expression contexts had to be updated. Moreover, in music-encoding#934, we agreed also to apply this rule to the manifestation context if the manifestation is not a singleton. This commit applies the necessary changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The schematron rule looks find to me, thanks @bwbohl! However, I noticed two things I think should be changed within the description:
- Not all elements affected by the schematron rule are listed within the description. I think it would be good to list all of them.
- My impression was that expression and manifestation were mixed up in the second half of the description – if I'm not totally mistaken there is no singleton expression.
The wording I provided is just a suggestion and I'm not set that it has to stay like this :)
Co-authored-by: Sophia Dörner <38356908+doerners@users.noreply.github.com>
thanks for the valuable contribution @doerners ! Merged it, If you would be so kind as to update your review ;-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bwbohl looks great!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even better!
As
<exhibHist>
,<treatHist>
and<treatSched>
have been remodelled after<provenance>
in #911, the corresponding Schematron rule to report an error in work and expression contexts had to be updated. Moreover, in #934, we agreed also to apply this rule to the manifestation context if the manifestation is not a singleton.This PR applies the necessary changes.