[GitHub] [apisix] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-02-07 Thread via GitHub
Fabriceli commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1421772278 @spacewander Could you request another committer to do the code review for this PR, thanks -- This is an automated message from the Apache Git Service. To respond to the message, pleas

[GitHub] [apisix] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-02-01 Thread via GitHub
Fabriceli commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1411819128 @tzssangglass could you have a look at this PR, thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [apisix] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-01-31 Thread via GitHub
Fabriceli commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1410044551 > > > [#8651 (comment)](https://github.com/apache/apisix/pull/8651#discussion_r1073064191) It seems there are still indentation issues. Some places use 8-spaces indentation instead of 4

[GitHub] [apisix] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-01-29 Thread via GitHub
Fabriceli commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1408104104 > [#8651 (comment)](https://github.com/apache/apisix/pull/8651#discussion_r1073064191) It seems there are still indentation issues. Some places use 8-spaces indentation instead of 4.

[GitHub] [apisix] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-01-19 Thread GitBox
Fabriceli commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1397953655 @spacewander Please take a look at this PR, thanks -- 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