[GitHub] [apisix-helm-chart] tokers commented on a change in pull request #27: add apisix-dashboard helm chart

2021-02-02 Thread GitBox
tokers commented on a change in pull request #27: URL: https://github.com/apache/apisix-helm-chart/pull/27#discussion_r568252751 ## File path: README.md ## @@ -19,63 +19,4 @@ # Apache APISIX Helm Charts -## Dependencies Review comment: Even so, link is necessary.

[GitHub] [apisix-helm-chart] tokers commented on a change in pull request #27: add apisix-dashboard helm chart

2021-02-02 Thread GitBox
tokers commented on a change in pull request #27: URL: https://github.com/apache/apisix-helm-chart/pull/27#discussion_r568582144 ## File path: README.md ## @@ -19,63 +19,4 @@ # Apache APISIX Helm Charts -## Dependencies Review comment: It's better to add links for

[GitHub] [apisix-helm-chart] tokers commented on a change in pull request #27: add apisix-dashboard helm chart

2021-02-01 Thread GitBox
tokers commented on a change in pull request #27: URL: https://github.com/apache/apisix-helm-chart/pull/27#discussion_r568253049 ## File path: README.md ## @@ -19,63 +19,4 @@ # Apache APISIX Helm Charts -## Dependencies Review comment: It's also required for dashbo

[GitHub] [apisix-helm-chart] tokers commented on a change in pull request #27: add apisix-dashboard helm chart

2021-02-01 Thread GitBox
tokers commented on a change in pull request #27: URL: https://github.com/apache/apisix-helm-chart/pull/27#discussion_r568252751 ## File path: README.md ## @@ -19,63 +19,4 @@ # Apache APISIX Helm Charts -## Dependencies Review comment: Even so, link is necessary.

[GitHub] [apisix-helm-chart] tokers commented on a change in pull request #27: add apisix-dashboard helm chart

2021-02-01 Thread GitBox
tokers commented on a change in pull request #27: URL: https://github.com/apache/apisix-helm-chart/pull/27#discussion_r567835088 ## File path: README.md ## @@ -19,63 +19,4 @@ # Apache APISIX Helm Charts -## Dependencies Review comment: Why removing these descriptio