Skip to content

fix(devtool): specify the host machine port used by explorer service#1415

Merged
Flouse merged 1 commit intoaxonweb3:mainfrom
sunchengzhu:main
Sep 11, 2023
Merged

fix(devtool): specify the host machine port used by explorer service#1415
Flouse merged 1 commit intoaxonweb3:mainfrom
sunchengzhu:main

Conversation

@sunchengzhu
Copy link
Copy Markdown
Contributor

What this PR does / why we need it?

The 4020 port of the Docker container is mapped to the 4020 port of the host machine.

What is the impact of this PR?

No Breaking Change

PR relation:

  • Ref #
CI Settings

CI Usage

Tip: Check the CI you want to run below, and then comment /run-ci.

CI Switch

  • E2E Tests
  • Web3 Compatible Tests
  • OCT 1-5 And 12-15
  • OCT 6-10
  • OCT 11
  • OCT 16-19
  • v3 Core Tests

CI Description

CI Name Description
Chaos CI Test the liveness and robustness of Axon under terrible network condition
Cargo Clippy Run cargo clippy --all --all-targets --all-features
Coverage Test Get the unit test coverage report
E2E Test Run end-to-end test to check interfaces
Code Format Run cargo +nightly fmt --all -- --check and cargo sort -gwc
Web3 Compatible Test Test the Web3 compatibility of Axon
v3 Core Test Run the compatibility tests provided by Uniswap V3
OCT 1-5 | 6-10 | 11 | 12-15 | 16-19 Run the compatibility tests provided by OpenZeppelin

@sunchengzhu sunchengzhu requested a review from a team as a code owner September 9, 2023 16:44
@KaoImin KaoImin changed the title fix(devtool):specify the host machine port used by the explorer container fix(devtool): specify the host machine port used by explorer container Sep 11, 2023
@Flouse Flouse changed the title fix(devtool): specify the host machine port used by explorer container fix(devtool): specify the host machine port used by explorer service Sep 11, 2023
@Flouse Flouse merged commit 10b8b15 into axonweb3:main Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants