Skip to content

Commit

Permalink
fix(ci): updated permissioning and versioning in GitHub Actions
Browse files Browse the repository at this point in the history
This fixes the failing 'deploy_docs.yaml' action by adding token with appropriate permissioning.
It pegs the protoc-gen-go-grpc version in the relevant actions to v1.4.0
to match what's written in the documentation
(this avoids test network setup failures when running integration tests.)
Also removed dead code from the Weaver Corda sample application.
Made 'mkdocs' publish URL customizable by owner.

Signed-off-by: VRamakrishna <[email protected]>
  • Loading branch information
VRamakrishna authored and sandeepnRES committed Aug 17, 2024
1 parent 30cde8b commit cd71082
Show file tree
Hide file tree
Showing 8 changed files with 10 additions and 10 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/deploy_docs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,15 @@ on:
env:
SITE_URL: https://${{ github.repository_owner }}.github.io/cacti
NODEJS_VERSION: v18.18.2
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}


jobs:
deploy-docs:
runs-on: ubuntu-22.04
permissions:
contents: write
pages: write
steps:
- name: Use Node.js ${{ env.NODEJS_VERSION }}
uses: actions/[email protected]
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_weaver-asset-exchange-fabric.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ jobs:
curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.6/protoc-3.15.6-linux-x86_64.zip
unzip protoc-3.15.6-linux-x86_64.zip -d protoc
go install google.golang.org/protobuf/cmd/protoc-gen-go@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.4.0
# PROTOS
- name: Build JS Protos
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_weaver-asset-transfer.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -728,7 +728,7 @@ jobs:
curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.6/protoc-3.15.6-linux-x86_64.zip
unzip protoc-3.15.6-linux-x86_64.zip -d protoc
go install google.golang.org/protobuf/cmd/protoc-gen-go@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.4.0
# PROTOS
- name: Build GO Protos
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/test_weaver-data-sharing.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -456,7 +456,7 @@ jobs:
unzip protoc-3.15.6-linux-x86_64.zip -d protoc
rm -rf protoc-3.15.6-linux-x86_64.zip
go install google.golang.org/protobuf/cmd/protoc-gen-go@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.4.0
- name: CI script for cleanup
run: ./tools/ci.sh
Expand Down Expand Up @@ -896,7 +896,7 @@ jobs:
curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.6/protoc-3.15.6-linux-x86_64.zip
unzip protoc-3.15.6-linux-x86_64.zip -d protoc
go install google.golang.org/protobuf/cmd/protoc-gen-go@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.4.0
# PROTOS
- name: Build GO Protos
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test_weaver-fabric-fabric-satp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ jobs:
curl -LO https://github.com/protocolbuffers/protobuf/releases/download/v3.15.6/protoc-3.15.6-linux-x86_64.zip
unzip protoc-3.15.6-linux-x86_64.zip -d protoc
go install google.golang.org/protobuf/cmd/protoc-gen-go@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@latest
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc@v1.4.0
# PROTOS GO
- name: Build GO Protos
Expand Down
2 changes: 1 addition & 1 deletion docs/mkdocs.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
site_name: Using and Developing with Hyperledger Cacti
site_url: !ENV [SITE_URL, 'https://hyperledger.github.io/cacti']
site_url: !ENV [SITE_URL, 'https://${{ github.repository_owner }}.github.io/cacti']
repo_name: hyperledger/cacti
repo_url: https://github.com/hyperledger/cacti
theme:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,6 @@ repositories {
dirs '../../../../core/network/corda-interop-app/interop-workflows/build/libs'
dirs '../../../../sdks/corda/build/libs'
dirs '../../../../common/protos-java-kt/build/libs'
dirs 'libs'
}
}
}
Expand Down
3 changes: 0 additions & 3 deletions weaver/samples/corda/corda-simple-application/makefile
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
CLIKT_VERSION=$(shell grep cliktVersion ./constants.properties | cut -d '=' -f 2)

.PHONY: build-local-weaver-dependencies
build-local-weaver-dependencies:
echo "Building local protos..."
Expand Down Expand Up @@ -112,4 +110,3 @@ publish-cordapps:
clean:
./gradlew clean
rm -rf .gradle
rm -rf clients/libs

0 comments on commit cd71082

Please sign in to comment.