samueleresca commented on PR #718:
URL: https://github.com/apache/pekko-connectors/pull/718#issuecomment-2207448067

   > @samueleresca the new #674 classes are breaking the doc build. See the 
checks for this PR.
   > 
   > It looks hard to fix. Paradox has built-in support for having 2 classes of 
the same name but one is in javadsl and one is in scaladsl. This support can't 
handle having a 2nd set of classes with the same name. The issue is if we try 
to use the FQCN, we will need to include the full package name and this negates 
the Paradox support for linking the javadsl/scaladsl version of the class 
depending on whether the UI user has chosen Java or Scala.
   > 
   > Would it be possible to rename all the `Jms` prefixed classes in the new 
connector to have something like `JakartaMs` or `JakartaMessaging` or 
`JakartaJms` prefix?
   
   Why the build wasn't catching this at PR level? (I haven't seen anything 
failing). Let me come back on this in the next few days. I can rename the 
classes and give it a try again. Sorry for the inconvenience


-- 
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: notifications-unsubscr...@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org
For additional commands, e-mail: notifications-h...@pekko.apache.org

Reply via email to