Skip to content

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented May 22, 2025

Minor change to fix the package configuration file in the Debian installation guide, which was incorrectly set to the RPM based file.

@shainaraskas : As the content came from a reused snippet we have lost the code formatting. Let me know your preference here. If you want to not use the snippet for this block of content or if we keep using it and we lose the code formatting.

If we use `{{pkg-conf}}` the substitution doesn't work.

I'm going to write a docs-builder issue to see if we can support inline code blocks with substitutions.

Closes #1430

@eedugon eedugon requested a review from a team as a code owner May 22, 2025 10:28
@eedugon eedugon added the Team:Admin Issues owned by the Admin Docs Team label May 22, 2025
Copy link
Collaborator

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

approved with a possible workaround to the code formatting thing you can take or leave

@eedugon eedugon enabled auto-merge (squash) May 23, 2025 08:49
@eedugon eedugon merged commit 9c5f8b8 into main May 23, 2025
5 of 6 checks passed
@eedugon eedugon deleted the fix_1430_debian_install branch May 23, 2025 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Admin Issues owned by the Admin Docs Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Issue]: /etc/sysconfig/elasticsearch file referenced in Debian install instructions

2 participants