Skip to content

Fix installation instructions for pgEdge on RHEL - #449

Open
zaidshabbir25 wants to merge 1 commit into
mainfrom
zaidshabbir25-patch-1
Open

Fix installation instructions for pgEdge on RHEL#449
zaidshabbir25 wants to merge 1 commit into
mainfrom
zaidshabbir25-patch-1

Conversation

@zaidshabbir25

Copy link
Copy Markdown
Contributor

Removed unnecessary 'dnf' installation command from prerequisites.

Removed unnecessary 'dnf' installation command from prerequisites.
@coderabbitai

coderabbitai Bot commented Jul 30, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 6b8d9921-29e0-44be-a6a8-c18937d23622

📥 Commits

Reviewing files that changed from the base of the PR and between 12fdcd5 and 8cc43b2.

📒 Files selected for processing (1)
  • docs/installation/systemd-installation.md

📝 Walkthrough

Walkthrough

The systemd installation guide updates its RPM prerequisites command to install only epel-release, removing the additional dnf package argument.

Changes

Systemd installation documentation

Layer / File(s) Summary
RPM prerequisite command
docs/installation/systemd-installation.md
The RPM prerequisites command now installs only epel-release.

Poem

I’m a rabbit with a tidy decree,
“Just one package is plenty for me!”
EPEL hops in place,
While dnf leaves no trace,
And the guide grows as neat as can be.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is missing the required Summary, Changes, Testing, Checklist, and Notes sections from the template. Add the template sections with a brief summary, bullet list of changes, testing steps, checklist items, and reviewer notes.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: fixing pgEdge installation instructions for RHEL.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch zaidshabbir25-patch-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@jason-lynch

Copy link
Copy Markdown
Member

@zaidshabbir25 As per your suggestion to link to the PEP repo docs, I've removed this part of the text in my PR: #448

That change will merge into main when we finalize the release. Could you please take a look and see if that addresses this issue?

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.

2 participants