You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/22 23:50:34 UTC

[GitHub] [pulsar] Shoothzj opened a new pull request #12143: [unit test] use correct line separator instead of \n

Shoothzj opened a new pull request #12143:
URL: https://github.com/apache/pulsar/pull/12143


   ### Motivation
   The unit test use \n, it can only paas in linux and OSX, can't run on windows.
   
   ### Modifications
   
   use System line separator instead of \n
   
   ### Documentation
   
   Check the box below and label this PR (if you have committer privilege).
   
   Need to update docs? 
     
   - [ ] no-need-doc 
     
   a little enhance, doesn't need doc
     
   
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] Shoothzj commented on pull request #12143: [unit test] use correct line separator instead of \n

Posted by GitBox <gi...@apache.org>.
Shoothzj commented on pull request #12143:
URL: https://github.com/apache/pulsar/pull/12143#issuecomment-925474872


   @merlimat @eolivelli PTAL


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] Shoothzj commented on pull request #12143: [unit test] use correct line separator instead of \n

Posted by GitBox <gi...@apache.org>.
Shoothzj commented on pull request #12143:
URL: https://github.com/apache/pulsar/pull/12143#issuecomment-925434799


   @sijie @codelipenghui @BewareMyPower PTAL


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] merlimat merged pull request #12143: [unit test] use correct line separator instead of \n

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #12143:
URL: https://github.com/apache/pulsar/pull/12143


   


-- 
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: commits-unsubscribe@pulsar.apache.org

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

[GitHub] [pulsar] Shoothzj closed pull request #12143: [unit test] use correct line separator instead of \n

Posted by GitBox <gi...@apache.org>.
Shoothzj closed pull request #12143:
URL: https://github.com/apache/pulsar/pull/12143


   


-- 
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: commits-unsubscribe@pulsar.apache.org

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