You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/12/08 09:00:33 UTC

[GitHub] [activemq-artemis] clebertsuconic commented on pull request #4308: Update ActiveMQSession.java

clebertsuconic commented on PR #4308:
URL: https://github.com/apache/activemq-artemis/pull/4308#issuecomment-1342296806

   - you need to squash all your commits into a single one, git push -f on your branch
   - you need to provide a test
   
   I'm not convinced there's anything broken. you need to provide a test as part of your commit.
   
   or if you are convinced there's an issue, you need to at least provide a way to reproduce and someone would take care of this issue and make a new PR with the test and the proper fix.


-- 
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: gitbox-unsubscribe@activemq.apache.org

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