Skip to content

Fix issue #1668: Provide a /packit help comment command#3069

Open
JackDKobe wants to merge 2 commits intopackit:mainfrom
JackDKobe:fix-issue-1668
Open

Fix issue #1668: Provide a /packit help comment command#3069
JackDKobe wants to merge 2 commits intopackit:mainfrom
JackDKobe:fix-issue-1668

Conversation

@JackDKobe
Copy link

Fixes good-first-issue #1668

@JackDKobe JackDKobe requested a review from a team as a code owner March 22, 2026 00:14
@JackDKobe JackDKobe requested review from betulependule and removed request for a team March 22, 2026 00:14
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request aims to add a /packit help command, but the only change submitted is the addition of tracking comments to the README.md file. The implementation for the new command appears to be missing from this pull request. Please add the relevant code changes for the feature to be reviewed. Regarding the changes in README.md, they include excessive newlines and a comment for an unrelated issue, which I've commented on separately.

Comment on lines +18 to +23


<!-- fix-issue-412: enable controlling packit-service with the command embedded -->


<!-- fix-issue-1668: Provide a /packit help comment command -->
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

These additions introduce several empty lines which makes the file less clean. Please consider removing them.

Also, the comment for issue #412 seems unrelated to this pull request. It might be better to handle that in a separate PR to keep changes focused.

@centosinfra-prod-github-app
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants