Add screenshot requirement

[koppor]

No description provided.

[qodo-free-for-open-source-projects]

Review Summary by Qodo

Add specific screenshot requirement to PR template

📝 Documentation

Walkthroughs

Description

• Add specific screenshot requirement to PR template
• Require screenshot showing library with single entry
• Entry must have submitter as author and issue number as title

Diagram

flowchart LR
  A["PR Template"] -- "Add screenshot requirement" --> B["Library entry screenshot"]
  B -- "Author: submitter" --> C["Issue number as title"]

Loading

File Changes

1. .github/PULL_REQUEST_TEMPLATE.md 📝 Documentation +1/-0

Add detailed screenshot requirement to checklist

• Added new checkbox item requiring screenshot of library with single entry
• Screenshot must show entry with submitter as author and issue number as title
• Clarifies existing screenshot requirement with specific example format

.github/PULL_REQUEST_TEMPLATE.md

---

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (1) 📎 Requirement gaps (0)

✅ 1. CI checkbox count mismatch 🐞 Bug ⛯ Reliability

Description

• The PR template now contains 7 checkbox lines under “### Mandatory checks”.
• The “Check PR Format” workflow enforces an exact checkbox-line count of 6, so PRs using the
  updated template will fail CI even if the author follows the template.
• This creates a contribution blocker (false-negative) and will require manual intervention or
  template rollback.

Code

.github/PULL_REQUEST_TEMPLATE.md[41]

+- [.] I added a screenshot in the PR description showing library with a single entry with me as author and as title the issue number.

Evidence

The PR template adds one more checklist line, but the workflow validation still hard-codes a
requirement of exactly 6 checkbox lines in the “Mandatory checks” section. Therefore, the newly
added line will cause the workflow to exit with failure when run against PR bodies following the
updated template.

.github/PULL_REQUEST_TEMPLATE.md[37-43]
.github/workflows/pr-format.yml[117-132]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The PR template adds a new mandatory checkbox line, but the GitHub Actions workflow `.github/workflows/pr-format.yml` still requires exactly 6 checkbox lines under `### Mandatory checks`. This will fail CI for otherwise-correct PRs.
### Issue Context
The workflow extracts checkbox lines (`grep "^- \["`) and enforces `LINE_COUNT != 6` as an error. The template now includes 7 checkbox lines.
### Fix Focus Areas
- .github/workflows/pr-format.yml[117-132]
- .github/PULL_REQUEST_TEMPLATE.md[37-43]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

✅ 2. Unclear screenshot checklist text 📘 Rule violation ✓ Correctness

Description

• The newly added checklist item contains awkward/incorrect grammar (e.g., missing article in
  showing library and unclear phrasing with me as author and as title the issue number).
• This reduces professionalism and can confuse contributors about the exact screenshot requirement.
• The template should be revised to a clear, grammatically correct instruction to meet documentation
  quality expectations.

Code

.github/PULL_REQUEST_TEMPLATE.md[41]

+- [.] I added a screenshot in the PR description showing library with a single entry with me as author and as title the issue number.

Evidence

PR Compliance ID 41 requires updated documentation/user-facing text to be free of obvious
typographical and grammatical errors. The added PR template line is user-facing guidance and
contains grammatically incorrect/unclear phrasing.

.github/PULL_REQUEST_TEMPLATE.md[41-41]
Best Practice: Learned patterns

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The added PR template checklist item is grammatically incorrect/unclear, which can confuse contributors and violates the requirement to avoid typos/grammar errors in documentation/user-facing text.
## Issue Context
This is a user-facing instruction in `.github/PULL_REQUEST_TEMPLATE.md`, so clarity and correctness matter.
## Fix Focus Areas
- .github/PULL_REQUEST_TEMPLATE.md[41-41]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

---

ⓘ The new review experience is currently in Beta. Learn more

[qodo-free-for-open-source-projects]

1. Ci checkbox count mismatch 🐞 Bug ⛯ Reliability

• The PR template now contains 7 checkbox lines under “### Mandatory checks”.
• The “Check PR Format” workflow enforces an exact checkbox-line count of 6, so PRs using the
  updated template will fail CI even if the author follows the template.
• This creates a contribution blocker (false-negative) and will require manual intervention or
  template rollback.

Agent Prompt

### Issue description
The PR template adds a new mandatory checkbox line, but the GitHub Actions workflow `.github/workflows/pr-format.yml` still requires exactly 6 checkbox lines under `### Mandatory checks`. This will fail CI for otherwise-correct PRs.

### Issue Context
The workflow extracts checkbox lines (`grep "^- \["`) and enforces `LINE_COUNT != 6` as an error. The template now includes 7 checkbox lines.

### Fix Focus Areas
- .github/workflows/pr-format.yml[117-132]
- .github/PULL_REQUEST_TEMPLATE.md[37-43]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

[testlens-app]

✅ All tests passed ✅

🏷️ Commit: c64283e
▶️ Tests: 11185 executed
⚪️ Checks: 53/53 completed

---

Learn more about TestLens at testlens.app.