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

rename light client config parameters #3740

Merged
merged 1 commit into from
Jun 14, 2022
Merged

rename light client config parameters #3740

merged 1 commit into from
Jun 14, 2022

Conversation

etan-status
Copy link
Contributor

For consistency with other options, use a common prefix for light client
data configuration options.

  • --serve-light-client-data --> --light-client-data-serve
  • --import-light-client-data --> --light-client-data-import-mode

No deprecation of the old identifiers as they were only sparingly used
and all usage can be easily updated without interferance.

For consistency with other options, use a common prefix for light client
data configuration options.

* `--serve-light-client-data` --> `--light-client-data-serve`
* `--import-light-client-data` --> `--light-client-data-import-mode`

No deprecation of the old identifiers as they were only sparingly used
and all usage can be easily updated without interferance.
@etan-status
Copy link
Contributor Author

Can be merged once status-im/infra-nimbus#107 is accepted

@zah zah marked this pull request as ready for review June 14, 2022 09:03
@zah zah merged commit 52ba4f7 into unstable Jun 14, 2022
@zah zah deleted the dev/etan/lc-conf branch June 14, 2022 09:03
etan-status added a commit that referenced this pull request Jun 14, 2022
…/dag-statevars

* commit '6bf7d6468036c3f010c6ce08727d63331ada5e0d':
  update CI timeouts (#3751)
  Fix /eth/v1/node/syncing (#3720)
  use block ID vs full block in LC data caching (#3741)
  cleanup LC data helpers (#3746)
  fix LC data import for Altair fork period (#3744)
  fix CI archive step (#3752)
  Support displaying the version number in the status bar; Implements #2959 (#3747)
  ci: install cmake on windows in daily job
  don't use slow-to-compress bzip2 with larger Jenkins artifacts
  rename light client config parameters (#3740)
  error handling for local testnet REST query (#3739)
  keep fcU consistent with actual DAG (#3748)
  remove unused `getExistingForkedBlock` overload (#3742)
  Version 22.5.2
  change proposer boost to 40%
  bump merge-testnets to get new ropsten TTD (#3668)
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