michael-o commented on PR #99: URL: https://github.com/apache/maven-doxia/pull/99#issuecomment-1139117634
> > @bertysentry Have a look at [apache/maven-doxia-sitetools@b002a41](https://github.com/apache/maven-doxia-sitetools/commit/b002a410296a248074eccd3a3af95b0ff7d697af). > > Ha ha! Must have been a lot of fun to update all these! To me everything looks in order. Yep, stupid trial and error. > You updated the _Parser_ classes to be consistent with the _Sink_ classes, that's even better than what I first suggested. You still need to check the **maven-site-plugin** integration tests. Yes, it strive for consistency and don't like surprises. Actually, I did run all MSITE ITs, they all pass. I have a local branch prepared with MSITE 4.x and upcoming Doxia/Doxia Sitetools. I do test the entire chain. > This is a nice change, well done! Thanks! Waiting for @hboutemy to go through as well. -- 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
