-
Notifications
You must be signed in to change notification settings - Fork 0
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
Refactor PhonemeTable as to make it maintainable #21
Conversation
looks like the pmd workflow is broken... interesting... edit: looks like pmd hasn't been updated here since 2022. whoops.. |
update pmd in open pull request
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.
PMD found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
…dcramer/Susquehanna into 19-refactor-phonemetablerefresh
Documentation is lacking; however there is an open issue specifically for it. As such, documentation will be added there. |
Fixes #19. Refactored PhonemeTable, so that it is no longer an unmaintainable mess. Also, fixed blank spacer cells from being marked as duplicates.