Skip to content

fix: point PR template contributing link at .github/CONTRIBUTING.md - #6

Merged
olitreadwell merged 1 commit into
mainfrom
fix/pr-template-contributing-link
Aug 19, 2026
Merged

fix: point PR template contributing link at .github/CONTRIBUTING.md#6
olitreadwell merged 1 commit into
mainfrom
fix/pr-template-contributing-link

Conversation

@olitreadwell

Copy link
Copy Markdown
Owner

Upstream target

themesberg/flowbite-react — no issue; self-found gap. The PR template links to a CONTRIBUTING.md that 404s.

What changed

  • Fixes the PR template's contributing-guide link: CONTRIBUTING.md#your-first-code-contribution (404) → .github/CONTRIBUTING.md#your-first-code-contribution (200).
  • Removes the root CONTRIBUTING.md that duplicated the existing .github/CONTRIBUTING.md (the two files were the same contributing-gen content).

Why

The template's first checkbox links to https://github.com/themesberg/flowbite-react/blob/main/CONTRIBUTING.md#your-first-code-contribution, which 404s because the guide actually lives at .github/CONTRIBUTING.md (verified: identical to the website page apps/web/content/docs/getting-started/contributing.mdx). GitHub does not redirect the root path to .github/, so the link path is simply wrong.

How to verify

  • curl -s -o /dev/null -w '%{http_code}' https://github.com/themesberg/flowbite-react/blob/main/.github/CONTRIBUTING.md#your-first-code-contribution returns 200.
  • The root CONTRIBUTING.md no longer exists; .github/CONTRIBUTING.md is unchanged.

Coverage

N/A — docs/link fix, no coverage tool applies.

Checks run locally

  • .github/CONTRIBUTING.md exists upstream and on the fork (sha edd224e), with the "Your First Code Contribution" section at the anchor.
  • New link path returns 200; old root path returns 404.
  • Root CONTRIBUTING.md removed; .github/CONTRIBUTING.md untouched.
  • Tests / Lint / Typecheck / Build: N/A — no code changed.

Style / template compliance

Follows the repo's PR template (checkbox above) and Conventional Commits commit style (fix: ...). One logical change: correct the link path.

Confidence: HIGH

Link path verified both ways (404 old, 200 new); the guide content already existed and is untouched.

AI assistance

Drafted with AI assistance. The gap was found and verified against the current repo (root link 404s, .github/CONTRIBUTING.md exists with the same content), the fix was verified with curl, and the diff was reviewed line by line before opening.

Recommendation

Ready to send upstream as-is.


PROMOTION NOTE (remove this section before/when opening against upstream):

gh pr create --repo themesberg/flowbite-react --base main --head olitreadwell:fix/pr-template-contributing-link

Prerequisites: none. No CLA/DCO bot found in themesberg/flowbite-react; no signed-commit requirement found.

The PR template links to CONTRIBUTING.md#your-first-code-contribution,
but the guide lives at .github/CONTRIBUTING.md (identical to the
website page apps/web/content/docs/getting-started/contributing.mdx),
so the root path 404s. Fix the link path and remove the root
CONTRIBUTING.md that duplicated the existing .github file.
@olitreadwell
olitreadwell merged commit e869818 into main Aug 19, 2026
5 checks passed
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.

1 participant