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

fix(config): optional block producer wallet for readonly node #768

Conversation

zeroqn
Copy link
Contributor

@zeroqn zeroqn commented Jul 20, 2022

Remove block producer wallet config requirement for readonly node

@zeroqn zeroqn marked this pull request as ready for review July 20, 2022 06:42
@zeroqn zeroqn requested review from jjyr, Flouse and blckngm July 20, 2022 06:42
@gw-bot
Copy link

gw-bot bot commented Jul 20, 2022

Running integration test

Workflow Run Id: 2702773569

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

failure

Flouse added a commit to godwokenrises/godwoken-docker-prebuilds that referenced this pull request Jul 20, 2022
@Flouse Flouse merged commit 5d11b3b into godwokenrises:develop Jul 20, 2022
Flouse added a commit to godwokenrises/godwoken-docker-prebuilds that referenced this pull request Jul 21, 2022
Flouse added a commit to godwokenrises/godwoken-docker-prebuilds that referenced this pull request Jul 21, 2022
Flouse added a commit to Flouse/godwoken-info that referenced this pull request Jul 21, 2022
- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768
Flouse added a commit to Flouse/godwoken-info that referenced this pull request Jul 21, 2022
- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768
jjyr pushed a commit to godwokenrises/godwoken-docker-prebuilds that referenced this pull request Jul 27, 2022
Flouse added a commit to Flouse/godwoken-info that referenced this pull request Jul 30, 2022
- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768
Flouse added a commit to godwokenrises/godwoken-info that referenced this pull request Jul 30, 2022
- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768
Flouse added a commit to godwokenrises/godwoken-info that referenced this pull request Aug 1, 2022
- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768
Flouse added a commit to godwokenrises/godwoken-info that referenced this pull request Aug 2, 2022
* mainnet_v1: rm useless mock.key

- fix(config): optional block producer wallet for readonly node
  godwokenrises/godwoken#768

* mainnet_v1: bump Godwoken to v1.4
1. remove useless `[block_producer.wallet_config]` in Godwoken readonly node
2. bump Godwoken to v1.4+
3. bump Godwoken Web3 to 1.5+
- https://github.com/nervosnetwork/godwoken/releases
- https://github.com/nervosnetwork/godwoken-scripts/releases
- https://github.com/nervosnetwork/godwoken-web3/releases
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.

4 participants