bisakhmondal commented on a change in pull request #1788:
URL: https://github.com/apache/apisix-dashboard/pull/1788#discussion_r634528665



##########
File path: api/build.sh
##########
@@ -32,13 +35,13 @@ set -x
 export ENV=local
 pwd=`pwd`
 
-rm -rf output && mkdir -p output/conf && mkdir -p output/dag-to-lua
+rm -rf output && mkdir -p output/conf && mkdir -p ./api/cmd/dag-to-lua

Review comment:
       for two reasons :) 
   1. we cant embed an empty directory.
   2. the embedded directory must be inside the package directory or 
subdirectory. for `// go:embed` to works.
   >Go source files that import "embed" can use the //go:embed directive to 
initialize a variable of type string, []byte, or FS with the contents of files 
read from the package directory or subdirectories at compile time.
   
   
   [reference](https://golang.org/pkg/embed/)




-- 
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:
[email protected]


Reply via email to