michael-o commented on PR #126:
URL: 
https://github.com/apache/maven-doxia-sitetools/pull/126#issuecomment-1886888874

   > > We always operate on parser modules consistently, thus I would expect 
that a parser config applies on a per-module basis.
   > 
   > You are probably right, a config mapping per id seems sufficient here.
   
   Yes
   
   > > The parser config isn't flexible
   > 
   > Right, but changing it would either require to add something like 
`Parser.configure(Map<String,String> config)` to Doxia which kind of conflicts 
with the existing setters or some sophisticated reflection to call the right 
setter. The tricky part is not to call the right setter method but to convert 
the parameter to the right type (leveraging the Plexus conversion logic). Which 
way of implementing this did you have in mind?
   
   About this, I need to think, but you are heading for that direction. I need 
more time. Can you make the first one happen as long as I think about the first 
one?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to