Skip to content
This repository has been archived by the owner on Feb 6, 2025. It is now read-only.

chore: add bsc and docker commands to readme #50

Merged
merged 3 commits into from
Jul 1, 2024

Conversation

forcodedancing
Copy link
Collaborator

@forcodedancing forcodedancing commented Jul 1, 2024

Description

Add bsc and dokcer commands to readme

Rationale

update readme

Example

NA

Changes

Notable changes:

  • readme

README.md Outdated
--http \
--http.addr=0.0.0.0 \
--http.port=8545 \
--http.api="admin, debug, eth, net, trace, txpool, web3, rpc, reth, ots, eth-call-bundle" \
Copy link
Collaborator

@unclezoro unclezoro Jul 1, 2024

Choose a reason for hiding this comment

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

what is this eth-call-bundle for? I think as an example, just expose necessary API modules, "admin" "debug" seems not suggested by default?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fixed

README.md Show resolved Hide resolved
@forcodedancing forcodedancing changed the title chore: add bsc commands to readme chore: add bsc and docker commands to readme Jul 1, 2024
@unclezoro unclezoro added this pull request to the merge queue Jul 1, 2024
Merged via the queue into bnb-chain:develop with commit 4256093 Jul 1, 2024
32 checks passed
pythonberg1997 added a commit to pythonberg1997/reth that referenced this pull request Jul 3, 2024
* ci: tag non-beta version as latest image (bnb-chain#51)

* chore: add bsc and docker commands to readme (bnb-chain#50)

* chore: add bsc commands to readme

* fix review comments

* fix op dockerfile issue

* fix: spec mapping issue and snapshot overwrite issue (bnb-chain#52)

* fix: system account status issue (bnb-chain#53)

* test1

* test2

* test3

* test4

* test5

* test6

* test7

* test8

* update CI

---------

Co-authored-by: dylanhuang <[email protected]>
Co-authored-by: forcodedancing <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants