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

Enhance template parsing error reporting #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ScraM-Team
Copy link
Sponsor Contributor

In a large project, it is hard to tell the source of errors in Flavour output, since filenames are missing in some cases.

This change provides more detail in error logs. Now HTML errors from jericho, and EL parsing errors include a filename. Example of the new output:

[INFO] Running TeaVM
Jun 06, 2020 7:09:31 PM net.htmlparser.jericho.LoggerProviderJava$JavaLogger error
SEVERE: **com.example.Client**: StartTag at (r6,c1,p144) missing required end tag
[INFO] Output file built with errors
[INFO] Debug information successfully written
[INFO] Source maps successfully written
[INFO] Source files successfully written
[ERROR] **templates/client.html**: Variable noSuchField was not found

The old output just said this, no filenames or classes:

[INFO] Running TeaVM
Jun 06, 2020 6:55:02 PM net.htmlparser.jericho.LoggerProviderJava$JavaLogger error
SEVERE: StartTag at (r6,c1,p144) missing required end tag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant