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
[DOXIATOOLS-80] Update to Doxia 2.0.0-M6 #41
Conversation
3c549f4
to
43c84f5
Compare
@michael-o Any idea why one XHTML5 test fails with a parsing exception in Doxia 2? |
<groupId>ch.qos.logback</groupId> | ||
<artifactId>logback-classic</artifactId> | ||
<version>1.2.11</version> | ||
</dependency> |
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.
Why not SLF4J simple with runtime scope?
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.
fixed
+ "' with the encoding '" + output.getEncoding() + "'"); | ||
} | ||
LOGGER.debug( | ||
"Parsing file from '{}' with the encoding '{}'" + " to '{}' with the encoding '{}'", |
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.
Why not make it one string now?
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.
fixed
if (debug) { | ||
log.setLogLevel(Log.LEVEL_DEBUG); | ||
// TODO: programmatically adjust log level | ||
} |
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.
Why then not drop the empty if
clause?
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.
fixed
I guess it misses the |
The generated XHTML is valid (look at the generated file) still the parser stumbles. |
only use index in sisu.plexus
No description provided.