Skip to content

docs(readme): point first-PR link to /contribute page#494

Merged
lefarcen merged 1 commit intonexu-io:mainfrom
lefarcen:fix/contribute-link
May 5, 2026
Merged

docs(readme): point first-PR link to /contribute page#494
lefarcen merged 1 commit intonexu-io:mainfrom
lefarcen:fix/contribute-link

Conversation

@lefarcen
Copy link
Copy Markdown
Contributor

@lefarcen lefarcen commented May 5, 2026

Summary

  • The good-first-issue label link in every README locale points to https://github.com/nexu-io/open-design/labels/good-first-issue, which renders an empty list whenever no open issue carries that label.
  • Switch the URL to https://github.com/nexu-io/open-design/contribute — GitHub's curated contribute page auto-pulls good-first-issue, help-wanted, and active items, so newcomers always land on something they can pick up.
  • Wording is unchanged across all 10 README locales (md, zh-CN, zh-TW, ja-JP, ko, de, fr, pt-BR, ru, uk); only the link target changes.

Test plan

  • Open each README on the PR and confirm the good-first-issue link points to /contribute.
  • Click through /contribute to confirm GitHub renders curated entries (rather than an empty label page).

The `labels/good-first-issue` URL renders an empty list when no issues
carry that label. GitHub's `/contribute` page auto-curates a mix of
good-first-issue, help-wanted, and active items, so it stays useful for
newcomers regardless of label coverage. Wording is unchanged across all
10 README locales.
@lefarcen lefarcen added the docs Documentation changes label May 5, 2026
Copy link
Copy Markdown
Contributor Author

@lefarcen lefarcen left a comment

Choose a reason for hiding this comment

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

Lens B review

The change is well-motivated — switching from (which shows empty when no issues carry that label) to (GitHub's curated first-contribution page) gives newcomers a better landing.

P3 minor inconsistency: The link text still says good-first-issue label but now opens GitHub's repository contribution page, not the label page directly. This could confuse readers who expect to land on the label search or want to understand what they're clicking.

Optional refinement: update the localized text to say "Contribute page" or "contribution page" (and mention it includes good first issue items), so the link text matches the target.

Not blocking — the current wording is acceptable for a small UX improvement. Just flagging for completeness.

Copy link
Copy Markdown
Contributor Author

@lefarcen lefarcen left a comment

Choose a reason for hiding this comment

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

Lens B review

The change is well-motivated — switching from /labels/good-first-issue (which shows empty when no issues carry that label) to /contribute (GitHub's curated first-contribution page) gives newcomers a better landing.

P3 minor inconsistency: The link text still says good-first-issue label but now opens GitHub's repository contribution page, not the label page directly. This could confuse readers who expect to land on the label search or want to understand what they're clicking.

Optional refinement: update the localized text to say "Contribute page" or "contribution page" (and mention it includes good first issue items), so the link text matches the target.

Not blocking — the current wording is acceptable for a small UX improvement. Just flagging for completeness.

Copy link
Copy Markdown
Contributor

@mrcfps mrcfps left a comment

Choose a reason for hiding this comment

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

@lefarcen I reviewed the README locale updates and verified the changed ranges consistently move the first-PR good-first-issue link from the empty label URL to GitHub's curated /contribute page across all ten README variants, without changing the surrounding localized wording. Nice focused docs fix — this should make the newcomer path more useful. 🙂

Generated by Looper 0.5.4 · runner=reviewer · agent=opencode

@lefarcen lefarcen merged commit 406a077 into nexu-io:main May 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants