Skip to content

Debian pkg config file updated in snippet #1457

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

Merged
merged 3 commits into from
May 23, 2025
Merged

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

Co-authored-by: shainaraskas <58563081+shainaraskas@users.noreply.github.com>
@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