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

Remove nonexistent option from enroll command #930

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Conversation

faec
Copy link
Contributor

@faec faec commented Feb 15, 2024

The synopsis for the enroll command listed --non-interactive, which is an option on the install command and doesn't apply to enroll.

The synopsis for the enroll command listed `--non-interactive`, which is an option on the install command and doesn't apply to enroll.
@faec faec requested a review from a team as a code owner February 15, 2024 21:31
Copy link

A documentation preview will be available soon.

Request a new doc build by commenting
  • Rebuild this PR: run docs-build
  • Rebuild this PR and all Elastic docs: run docs-build rebuild

run docs-build is much faster than run docs-build rebuild. A rebuild should only be needed in rare situations.

If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here.

Copy link
Contributor

@kilfoyle kilfoyle 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 for this @faec. I'll look after merging and will babysit the backports to make sure the change finds its way into the current docs.

@kilfoyle kilfoyle merged commit e4772c2 into 8.12 Feb 15, 2024
kilfoyle pushed a commit to kilfoyle/ingest-docs that referenced this pull request Feb 15, 2024
The synopsis for the enroll command listed `--non-interactive`, which is an option on the install command and doesn't apply to enroll.

(cherry picked from commit e4772c2)

# Conflicts:
#	docs/en/ingest-management/commands.asciidoc
@kilfoyle
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.13

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kilfoyle pushed a commit to kilfoyle/ingest-docs that referenced this pull request Feb 15, 2024
The synopsis for the enroll command listed `--non-interactive`, which is an option on the install command and doesn't apply to enroll.

(cherry picked from commit e4772c2)

# Conflicts:
#	docs/en/ingest-management/commands.asciidoc
@kilfoyle
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
main

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kilfoyle added a commit that referenced this pull request Feb 15, 2024
The synopsis for the enroll command listed `--non-interactive`, which is an option on the install command and doesn't apply to enroll.

(cherry picked from commit e4772c2)

# Conflicts:
#	docs/en/ingest-management/commands.asciidoc

Co-authored-by: Fae Charlton <fae.charlton@elastic.co>
kilfoyle added a commit that referenced this pull request Feb 15, 2024
The synopsis for the enroll command listed `--non-interactive`, which is an option on the install command and doesn't apply to enroll.

(cherry picked from commit e4772c2)

# Conflicts:
#	docs/en/ingest-management/commands.asciidoc

Co-authored-by: Fae Charlton <fae.charlton@elastic.co>
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.

2 participants