[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-04-16 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-821081464


   > > Hi @vinayaksh42, do you need any help from the community?
   > 
   > Hey, I am not sure about how to write the test
   
   Can you give more details about it? 
   So that I can help you solve it.
   
   Maybe you can refer 
https://github.com/apache/apisix-dashboard/blob/master/api/test/shell/cli_test.sh.


-- 
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

[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-02-22 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-783790069


   > > Hi @vinayaksh42, do you meet any problem?
   > > I think you should merge the master branch.
   > 
   > Could you help to sync codes with this PR?
   
   Sure, my pleasure.



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

[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-02-21 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-782861106


   Hi @vinayaksh42, do you meet any problem?
   I think you should merge the master branch.



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

[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-02-06 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-774574324


   Oh sorry.@vinayaksh42 You need to merge the master branch again.



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

[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-02-05 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-773818747


   Hi, @vinayaksh42
   Maybe you should fix the conflict and sync the master branch.



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

[GitHub] [apisix-dashboard] starsz commented on pull request #1420: feat: added support for -c to set the config file path

2021-02-04 Thread GitBox 


starsz commented on pull request #1420:
URL: https://github.com/apache/apisix-dashboard/pull/1420#issuecomment-773818747


   Hi, @vinayaksh42
   Maybe you should fix the conflict and sync the master branch.



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