satish-csi commented on pull request #119:
URL: https://github.com/apache/commons-vfs/pull/119#issuecomment-792324602


   > > @garydgregory This needs to be merged into master! This issue can be 
reproduced but requires infra for writing testcases hence I am unable to write 
test cases.
   > 
   > @satish-csi
   > As we are volunteer-based organization, may I suggest that you would be 
better served in more diplomatic engagement with the community instead of 
issuing commands like "This needs to be merged into master!"
   > 
   > If you want any PR to get more attention, providing tests is the way to 
go. We have a lot of existing tests you are free to use as inspiration, 
specifically in the HTTP provider packages:
   > 
   > * org.apache.commons.vfs2.provider.http
   > * org.apache.commons.vfs2.provider.http4
   > * org.apache.commons.vfs2.provider.http4s
   > * org.apache.commons.vfs2.provider.http5
   > * org.apache.commons.vfs2.provider.http5s
   > * org.apache.commons.vfs2.provider.https
   
   Hi @garydgregory 
   I am extremely sorry for making incorrect statement, I was expecting little 
hint on how to write testcase for infrastructure dependencies, but no worries 
will dig more into code, 
   I apologise for 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.

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


Reply via email to