Skip to content

Commit

Permalink
Merge pull request #1130 from scrtlabs/overhaul-protobuf
Browse files Browse the repository at this point in the history
Overhaul gRPC queries
  • Loading branch information
assafmo authored Sep 6, 2022
2 parents 0b323ec + 7c40dd6 commit c1480d3
Show file tree
Hide file tree
Showing 24 changed files with 3,618 additions and 2,185 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,7 @@ jobs:
make kill-localsecret # next step needs the localsecret ports
- name: Run secret.js tests
run: |
git clone --depth 1 --branch cosmwasm-v1 https://github.com/scrtlabs/secret.js
git clone --depth 1 --branch cw1-overhaul-protobuf https://github.com/scrtlabs/secret.js
cd secret.js
# Use the docker images that we built just a few steps above
perl -i -pe 's/localsecret:.+?"/localsecret:v0.0.0"/' ./test/*
Expand Down
25 changes: 6 additions & 19 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -435,7 +435,7 @@ aesm-image:
docker build -f deployment/dockerfiles/aesm.Dockerfile -t enigmampc/aesm .

###############################################################################
### Swagger ###
### Swagger & Protobuf ###
###############################################################################

# Install the runsim binary with a temporary workaround of entering an outside
Expand All @@ -456,22 +456,12 @@ update-swagger-openapi-docs: statik proto-swagger-openapi-gen
echo "\033[92mSwagger docs are in sync\033[0m";\
fi

.PHONY: update-swagger-openapi-docs statik

###############################################################################
### Protobuf ###
###############################################################################

## proto-all: proto-gen proto-lint proto-check-breaking

# proto-gen:
# @./scripts/protocgen.sh
# Example `CHAIN_VERSION=v1.4 make proto-swagger-openapi-gen`
proto-swagger-openapi-gen:
@./scripts/protoc-swagger-openapi-gen.sh

# proto-lint:
# @buf check lint --error-format=json
.PHONY: update-swagger-openapi-docs statik proto-swagger-openapi-gen

# proto-check-breaking:
# @buf check breaking --against-input '.git#branch=master'

protoVer=v0.2

Expand All @@ -481,10 +471,7 @@ proto-gen:
@echo "Generating Protobuf files"
$(DOCKER) run --rm -v $(CURDIR):/workspace --workdir /workspace tendermintdev/sdk-proto-gen:$(protoVer) sh ./scripts/protocgen.sh

proto-swagger-openapi-gen:
@./scripts/protoc-swagger-openapi-gen.sh

proto-lint:
@$(DOCKER_BUF) lint --error-format=json

.PHONY: proto-all proto-gen proto-swagger-openapi-gen proto-format proto-lint proto-check-breaking
.PHONY: proto-all proto-gen proto-format proto-lint proto-check-breaking
4 changes: 2 additions & 2 deletions client/docs/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"info": {
"title": "Secret Network",
"description": "A REST interface for queries and transactions",
"version": "v1.4.0"
"version": "v1.4"
},
"apis": [
{
Expand Down Expand Up @@ -243,4 +243,4 @@
}
}
]
}
}
30 changes: 20 additions & 10 deletions client/docs/static/openapi/index.html

Large diffs are not rendered by default.

Loading

0 comments on commit c1480d3

Please sign in to comment.