Based on feedback of @soamvasani, the instructions for compiling are too ambiguous. Since writing the instructions the build/wfcli directory was added, causing users that followed the compiling.md instructions literally to hit the following error when building:
go install github.com/fission/fission-workflows/cmd/wfcli: build output "wfcli" already exists and is a directory
This error is caused because the build artifact (wfcli) conflicts with the build/wfcli when building from the build/ directory. Building from any other directory would not cause this error.
To fix this issue, I...
renamed build/wfcli to build/wfcli-docker to avoid the naming conflict
updated the compiling.md instructions to reflect the current state and also be more thorough in explaining the options.
Based on feedback of @soamvasani, the instructions for compiling are too ambiguous. Since writing the instructions the
build/wfclidirectory was added, causing users that followed thecompiling.mdinstructions literally to hit the following error when building:This error is caused because the build artifact (
wfcli) conflicts with thebuild/wfcliwhen building from thebuild/directory. Building from any other directory would not cause this error.To fix this issue, I...
build/wfclitobuild/wfcli-dockerto avoid the naming conflictcompiling.mdinstructions to reflect the current state and also be more thorough in explaining the options.