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

Improve canister network setup #2068

Merged
merged 1 commit into from
Nov 21, 2023
Merged

Improve canister network setup #2068

merged 1 commit into from
Nov 21, 2023

Conversation

nmattia
Copy link
Collaborator

@nmattia nmattia commented Nov 21, 2023

This improves the way we spin up canisters in a couple of ways:

  • The dfx.json file is made actually useful by creating & deploying all the test canisters
  • By default II is deployed (through dfx.json) using the test options (fetching the root key & using the dummy captcha). Note: as far as I can tell the dfx.json isn't used to deploy anything to mainnet.
  • The chrome options default to something useful locally (no headless)
  • The HACKING instructions are updated & simplified

This creates a more sane setup where all canisters are deployed in .dfx, with simpler commands.

This improves the way we spin up canisters in a couple of ways:

* The `dfx.json` file is made actually useful by creating & deploying
  all the test canisters
* By default II is deploy (through dfx.json) using the test options
  (fetching the root key & using the dummy captcha). Note: as far as I
  can tell the `dfx.json` isn't used to deploy anything to mainnet.
* The chrome options default to something useful locally (no headless)
* The HACKING instructions are updated & simplified

This creates a more sane setup where all canisters are deployed in
`.dfx`, with simpler commands.
Copy link
Contributor

@frederikrothenberger frederikrothenberger left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@nmattia nmattia added this pull request to the merge queue Nov 21, 2023
Merged via the queue into main with commit fde0322 Nov 21, 2023
53 checks passed
@nmattia nmattia deleted the nm-dfx-useful branch November 21, 2023 14:57
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