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

feat: bee start cmd starts the node in the ultra-light mode #4326

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

mrekucci
Copy link
Contributor

@mrekucci mrekucci commented Sep 20, 2023

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

This changes the bee start cmd default behavior to start the node in the ultra-light mode.

Closes
#4114
#3756

@mrekucci mrekucci changed the title feat: bee start starts the node in the ultra-light mode feat: bee start cmd starts the node in the ultra-light mode Sep 20, 2023
@istae
Copy link
Member

istae commented Sep 20, 2023

do we also need to change the default values for the config options?

@mrekucci mrekucci force-pushed the ultralight-by-default branch from 44bf5a5 to f586c9f Compare September 20, 2023 11:48
@mrekucci
Copy link
Contributor Author

do we also need to change the default values for the config options?

Good catch! Just for the swap-enable, the others are fine. Fixed.

@istae
Copy link
Member

istae commented Sep 20, 2023

just to confirm, did you manually run the cmd to confirm that it boots with ultra-light mode?

@mrekucci
Copy link
Contributor Author

just to confirm, did you manually run the cmd to confirm that it boots with ultra-light mode?

Yes!

@mrekucci mrekucci merged commit ca78646 into master Sep 20, 2023
@mrekucci mrekucci deleted the ultralight-by-default branch September 20, 2023 12:30
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