Skip to content

Conversation

leemthompo
Copy link
Contributor

We want to refer to the README as the source of truth, so keeping this page as lean as possible

https://github.com/elastic/search-team/issues/9677#issuecomment-2765378363

We want to refer to the README as the source of truth, so keeping this page as lean as possible

elastic/search-team#9677 (comment)
@leemthompo leemthompo self-assigned this Mar 31, 2025
@leemthompo leemthompo requested a review from a team March 31, 2025 10:25
Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

👨‍💻

@leemthompo leemthompo enabled auto-merge (squash) March 31, 2025 10:50
@leemthompo leemthompo merged commit 3cc37ed into main Mar 31, 2025
3 of 4 checks passed
@leemthompo leemthompo deleted the leemthompo-patch-2 branch March 31, 2025 10:51
Quickly set up {{es}} and {{kib}} in Docker for local development or testing, using the [`start-local` script](https://github.com/elastic/start-local?tab=readme-ov-file#-try-elasticsearch-and-kibana-locally).
Quickly set up {{es}} and {{kib}} in Docker for local development or testing, using this one-liner in the command line.

This setup comes with a one-month trial license that includes all Elastic features. After the trial period, the license reverts to **Free and open - Basic**. Refer to [Elastic subscriptions](https://www.elastic.co/subscriptions) for more information.
Copy link
Contributor

Choose a reason for hiding this comment

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

@leemthompo I wonder if we should continue including the subscriptions link? Was this just trimming off things, or do you have a reason to remove this? (Agree I think we can trim the line down—so maybe just a very simple call-to-action with the subscription link?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

as Enrico mentioned, we're relying on the README as the source of truth, so we can just defer to https://github.com/elastic/start-local/blob/main/README.md#-features

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it's more about the ratio of signal-to-noise on the page than anything else

start-local is a one-line command and we need to remove everything we can to not obfuscate the actual instructions on the page

we've got a big "don't use in production" warning and links to the readme for full details already, so I think this edit makes sense

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