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

Fix Command Flag Typos and Syntax Issues #14758

Merged
merged 3 commits into from
Dec 30, 2024

Conversation

savvar9991
Copy link
Contributor

1.Changed "file and" to "file an:

  • outdated interop guides or instructions. If you experience issues with this guide, please file and
  • outdated interop guides or instructions. If you experience issues with this guide, please file an

2.Changed "distinguish" to "distinguish:

  • "--" is a flag divider to distinguish between bazel flags and flags that should be passed to the application. All flags and
  • "--" is a flag divider to distinguish between bazel flags and flags that should be passed to the application. All flags and

3.Changed "interopt" to "interop:

  • bazel run //cmd/validator --config=minimal -- --datadir=/tmp/validator --interopt-num-validators=256 --minimal-config --suggested-
  • bazel run //cmd/validator --config=minimal -- --datadir=/tmp/validator --interop-num-validators=256 --minimal-config --suggested

This fixes the command flag name. "interopts" was a typo, the correct flag name is "interop" (short for "interoperability"). This is a critical fix because an incorrect flag name would cause the command to fail when executed.

@savvar9991 savvar9991 requested a review from a team as a code owner December 29, 2024 09:58
@CLAassistant
Copy link

CLAassistant commented Dec 29, 2024

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@rkapka rkapka left a comment

Choose a reason for hiding this comment

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

Approving because a command got fixed apart from regular typos

@rkapka rkapka enabled auto-merge December 30, 2024 12:24
@rkapka rkapka added this pull request to the merge queue Dec 30, 2024
Merged via the queue into prysmaticlabs:develop with commit f55e622 Dec 30, 2024
14 of 15 checks passed
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.

3 participants