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

800+ priority errors that aren't really errors #198

Open
birchamp opened this issue Dec 13, 2023 · 3 comments
Open

800+ priority errors that aren't really errors #198

birchamp opened this issue Dec 13, 2023 · 3 comments
Assignees
Labels
bug Something isn't working Priorty/High High priority issue

Comments

@birchamp
Copy link
Collaborator

Currently there are some errors that are unnecessary. @richmahn should know which numbers are invalid.

@birchamp birchamp added bug Something isn't working Priorty/High High priority issue labels Dec 13, 2023
@richmahn
Copy link
Member

This is from the latest SNG (Song of Songs) BP:

image

From ZEP:

image

Note: Some error my be valid, but very confusing if it is really 800 level (will crash tC/tCC/GWE or not) and/or more info on what should be fixed.

Also, the error (whatever level it is) about the "Quote" field shouldn't have & in it should be removed, as it is allowed.

Another one, now fixed in ZEP, was this about having a \q# marker not starting at the beginning of the line. Is that really a 800 level error?

image

@richmahn
Copy link
Member

richmahn commented Dec 13, 2023

A few more. Verse bridges don't seem to be supported:

image

More \q# and \p# (are \p# supported?) errors:

image

@richmahn
Copy link
Member

Another validation that should be done:

if a \fqa appears, it should look to see if there is a space before it. Otherwise the text before and after \fqa will be connected, no space.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priorty/High High priority issue
Projects
None yet
Development

No branches or pull requests

3 participants