Skip to content

Commit

Permalink
Merge pull request #479 from uber/edwin.http
Browse files Browse the repository at this point in the history
Run easyjson for any module type with http
  • Loading branch information
edwinzg committed Oct 4, 2018
2 parents 8f02333 + b9ef363 commit 59e39cf
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
6 changes: 6 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,12 @@ git clone git@github.com:uber/zanzibar $GOPATH/src/github.com/uber/zanzibar
cd $GOPATH/src/github.com/uber/zanzibar
make install
```
### Running make generate
```
make generate
```
If this fails due to `yq: command not found`, then install `yq` by running `pip install yq`.
If you have already installed `yq` and the build fails due to `unknown command "examples/example-gateway/clients/baz/client-config.yaml" for "yq"` then delete `yq` and reinstall by running `pip install --force-reinstall yq`. This is because Zanzibar uses the python distribution of `yq` so other distributions of `yq` will not work.

### Running the tests

Expand Down
2 changes: 1 addition & 1 deletion codegen/runner/pre-steps.sh
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ for config_file in ${config_files}; do
fi

module_type=$($processor -r .type "$config_file")
[[ ${module_type} != "http" ]] && continue
[[ ${module_type} != *"http"* ]] && continue
dir=$(dirname "$config_file")
yaml_files=$(find "$dir" -name "*.json" -o -name "*.yaml")
for yaml_file in ${yaml_files}; do
Expand Down

0 comments on commit 59e39cf

Please sign in to comment.