Skip to content

Conversation

kirisakow
Copy link

fix #437

@cla-checker-service
Copy link

cla-checker-service bot commented Feb 13, 2025

💚 CLA has been signed

@eedugon
Copy link
Contributor

eedugon commented Apr 10, 2025

good catch, thanks @kirisakow !

@eedugon eedugon self-requested a review April 10, 2025 05:50
Copy link
Contributor

@eedugon eedugon left a comment

Choose a reason for hiding this comment

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

LGTM!

@kirisakow kirisakow closed this by deleting the head repository Apr 28, 2025
eedugon added a commit that referenced this pull request Apr 30, 2025
Substitutes #438

Closes #437

It fixes a couple of wrong command execution paths and it also
consolidates the command execution examples to `./bin/elasticsearch....`
instead of `bin/elasticsearch...` (using ./ is safer in windows power
shell and it's totally valid in Linux).

Also `./bin` is already used for a lot linux examples like starting
Elasticsearch from the command line.
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.

wrong path: C:\...\bin>bin\...

2 participants