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.

Sign up for GitHub

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

Jump to bottom

Fixed: Rework tied notes #90

Merged
merged 1 commit into from
May 23, 2024
Merged

Fixed: Rework tied notes #90

merged 1 commit into from
May 23, 2024

Conversation

ahankinson
Copy link
Contributor

  • Clarifies the use of ties
  • Restricts them to only occur on notes

Fixes #1
Fixes #88
Refs #28

@ahankinson
Fixed: Rework tied notes
9602d22 
- Clarifies the use of ties
- Restricts them to only occur on notes

Fixes #1
Fixes #88
Refs #28
@ahankinson ahankinson requested a review from lpugin May 23, 2024 11:46
@ahankinson ahankinson added the Type: Spec Change Changes to the PAE Specification label May 23, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented May 23, 2024 •
edited
Loading

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-05-23 12:46 UTC

lpugin
lpugin approved these changes May 23, 2024
View reviewed changes
Copy link
Contributor

@lpugin lpugin left a comment

Choose a reason for hiding this comment

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

We will revise this once we have clarified chords.

@lpugin lpugin merged commit 2b9363a into main May 23, 2024
1 check passed
@lpugin lpugin deleted the fixed-tied-notes branch May 23, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lpugin lpugin lpugin approved these changes

Assignees
No one assigned
Labels
Type: Spec Change Changes to the PAE Specification
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Disallow rests within ties Use underscore instead of plus for tied notes
2 participants
@ahankinson @lpugin