Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for RC6 chain and GSRPC versions #1439

Merged
merged 20 commits into from
Oct 6, 2020

Conversation

mikiquantum
Copy link
Contributor

@mikiquantum mikiquantum commented Sep 30, 2020

No description provided.

@@ -148,13 +148,6 @@ type Events struct {
ChainBridge_RelayerAdded []EventRelayerAdded //nolint:stylecheck,golint
ChainBridge_RelayerThresholdChanged []EventRelayerThresholdChanged //nolint:stylecheck,golint
Nfts_DepositAsset []EventNFTDeposited //nolint:stylecheck,golint
Council_Proposed []types.EventCollectiveProposed //nolint:stylecheck,golint
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already part of GSRPC events

@@ -1,7 +1,7 @@
version: '3'
services:
cc:
image: "centrifugeio/centrifuge-chain:20200625211147-5e0e5e1"
image: "centrifugeio/centrifuge-chain:20200928161532-a960a81"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

RC6 image

@mikiquantum mikiquantum added the WIP in progress label Sep 30, 2020
@codecov
Copy link

codecov bot commented Oct 5, 2020

Codecov Report

Merging #1439 into develop will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1439      +/-   ##
===========================================
- Coverage    66.94%   66.92%   -0.03%     
===========================================
  Files          167      167              
  Lines        10613    10614       +1     
===========================================
- Hits          7105     7103       -2     
- Misses        2739     2742       +3     
  Partials       769      769              
Impacted Files Coverage Δ
centchain/api.go 62.85% <100.00%> (+0.26%) ⬆️
utils/byteutils/bytes.go 79.77% <0.00%> (-3.38%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6fb09ed...e19b0a3. Read the comment docs.

@mikiquantum mikiquantum removed the WIP in progress label Oct 5, 2020
@vedhavyas
Copy link
Contributor

@mikiquantum looks like there is another PR with substrate v2.0.0 changes centrifuge/go-substrate-rpc-client#100. Is it worth the time to wait for that to get in and update this accordingly ?

@mikiquantum
Copy link
Contributor Author

@vedhavyas I would not, since that version might not be compatible with our chain.

@mikiquantum mikiquantum merged commit 7b26e2e into centrifuge:develop Oct 6, 2020
@mikiquantum mikiquantum deleted the up/rc6 branch October 6, 2020 14:39
mikiquantum added a commit that referenced this pull request Oct 13, 2020
This reverts commit 7b26e2e.

	modified:   .travis.yml
	modified:   build/scripts/docker/docker-compose-bridge.yml
	modified:   build/scripts/docker/docker-compose-cc.yml
	deleted:    build/scripts/test-dependencies/bridge/config/config.example.json
	new file:   build/scripts/test-dependencies/bridge/config/config.example.toml
	modified:   build/scripts/test-dependencies/bridge/create_config.sh
	modified:   centchain/api.go
	modified:   centchain/events.go
	modified:   go.mod
	modified:   go.sum
	modified:   resources/data.go
mikiquantum added a commit that referenced this pull request Oct 13, 2020
* Use temp fix for indices

* Revert "Fix Docker subkey (#1440)"

This reverts commit 2381280.

	modified:   Dockerfile

* Revert "Update for RC6 chain and GSRPC versions (#1439)"

This reverts commit 7b26e2e.

	modified:   .travis.yml
	modified:   build/scripts/docker/docker-compose-bridge.yml
	modified:   build/scripts/docker/docker-compose-cc.yml
	deleted:    build/scripts/test-dependencies/bridge/config/config.example.json
	new file:   build/scripts/test-dependencies/bridge/config/config.example.toml
	modified:   build/scripts/test-dependencies/bridge/create_config.sh
	modified:   centchain/api.go
	modified:   centchain/events.go
	modified:   go.mod
	modified:   go.sum
	modified:   resources/data.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants