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

refactor(cli): initialize chain without any secret key #1467

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

yangby-cryptape
Copy link
Collaborator

@yangby-cryptape yangby-cryptape commented Oct 10, 2023

What this PR does / why we need it?

This PR remove the key parameter from the init sub-command.

Since #1454, the secret is not required anymore.

What is the impact of this PR?

No Breaking Change

CI Settings

CI Usage

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

CI Switch

  • 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
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

@yangby-cryptape

This comment was marked as off-topic.

@github-actions

This comment was marked as outdated.

@yangby-cryptape yangby-cryptape force-pushed the yangby/refactor/no-key-when-init branch from 81cfc98 to fddc5e3 Compare October 11, 2023 03:05
@yangby-cryptape yangby-cryptape changed the title refactor(cli): initialize chain without secret keys refactor(cli): initialize chain without any secret key Oct 11, 2023
@yangby-cryptape

This comment was marked as off-topic.

@github-actions

This comment was marked as outdated.

@yangby-cryptape yangby-cryptape marked this pull request as ready for review October 11, 2023 03:06
@yangby-cryptape yangby-cryptape requested a review from a team as a code owner October 11, 2023 03:06
@yangby-cryptape yangby-cryptape requested review from ashuralyk and wenyuanhust and removed request for ashuralyk and wenyuanhust October 11, 2023 03:06
@yangby-cryptape yangby-cryptape force-pushed the yangby/refactor/no-key-when-init branch from fddc5e3 to 45aca20 Compare October 11, 2023 03:12
@yangby-cryptape

This comment was marked as off-topic.

@github-actions

This comment was marked as outdated.

@yangby-cryptape yangby-cryptape force-pushed the yangby/refactor/no-key-when-init branch from 45aca20 to 0bfd3e8 Compare October 11, 2023 03:24
@yangby-cryptape
Copy link
Collaborator Author

⚠️ The following CI checks could NOT be passed unless this PR merged.

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

Because only the workflow scripts in default branch will be called.

@Flouse Flouse requested review from KaoImin and driftluo October 11, 2023 04:01
@KaoImin KaoImin added this pull request to the merge queue Oct 11, 2023
Merged via the queue into main with commit c72bf9c Oct 11, 2023
17 of 20 checks passed
@KaoImin KaoImin deleted the yangby/refactor/no-key-when-init branch October 12, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants