-
Notifications
You must be signed in to change notification settings - Fork 3
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
👷 Added PR checklist workflow #824
👷 Added PR checklist workflow #824
Conversation
WalkthroughA new GitHub Actions workflow ( Changes
Possibly related PRs
Suggested reviewers
Poem
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
…ionality-via-github-action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (3)
.github/workflows/pr-checklist.yml
(1 hunks)docs/develop.md
(1 hunks)docs/organize.md
(1 hunks)
🧰 Additional context used
🪛 YAMLlint (1.35.1)
.github/workflows/pr-checklist.yml
[warning] 3-3: truthy value should be one of [false, true]
(truthy)
[error] 11-11: no new line character at the end of file
(new-line-at-end-of-file)
🪛 LanguageTool
docs/develop.md
[style] ~217-~217: This phrase is redundant. Consider using “inside”.
Context: ...for validating the status of checklists inside of a PR description. To be able to merge a...
(OUTSIDE_OF)
[uncategorized] ~217-~217: Possible missing comma found.
Context: ...a PR description. To be able to merge a PR all checklist items have to be ticked o...
(AI_HYDRA_LEO_MISSING_COMMA)
docs/organize.md
[style] ~55-~55: As a shorter alternative for ‘able to’, consider using “can”.
Context: ...tml#require-pr-checklist) the templates are able to enforce a careful examination of each P...
(BE_ABLE_TO)
🔇 Additional comments (2)
.github/workflows/pr-checklist.yml (2)
1-2
: Clear Workflow Name Declaration.
The workflow name "Require PR Checklist" clearly communicates its intent.
3-6
: Well-Defined Trigger Events.
The specified pull request events (opened, edited, reopened, synchronize, ready_for_review) adequately cover the scenarios when the checklist validation is needed.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 3-3: truthy value should be one of [false, true]
(truthy)
…ionality-via-github-action
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (2)
docs/develop.md
(1 hunks)docs/organize.md
(1 hunks)
🧰 Additional context used
🪛 GitHub Actions: Deploy documentation website
docs/develop.md
[warning] 1-1: Code style issues found in the above file. Run Prettier with --write to fix.
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build (refarch-backend)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
📜 Review details
Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (1)
docs/develop.md
(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: build (refarch-backend)
🔇 Additional comments (1)
docs/develop.md (1)
219-220
: Verify the PR Template Link
The section includes a link: “PR template”. Please confirm that the target reference is correct. If the documentation is written in Markdown, verify whether the link should use a “.md” extension rather than “.html” to ensure consistency and proper navigation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Pull Request
Changes
Reference
Issue: #803
Checklist
Note: If some checklist items are not relevant for your PR, just remove them.
General
I have read the Contribution Guidelines (TBD)Summary by CodeRabbit