Skip to content

docs: move all-contributors roster to shared profile - #83

Merged
trvny merged 4 commits into
mainfrom
docs/centralize-all-contributors-20261113
Nov 13, 2026
Merged

trvny merged 4 commits into
mainfrom
docs/centralize-all-contributors-20261113

Conversation

@trvny

@trvny trvny commented Nov 13, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Move the 31-entry All Contributors roster from trvny/trvny to the shared profile README.
  • Add .all-contributorsrc here as the canonical configuration (projectName: .github).
  • Keep original activity links in the existing table; the generated cross-repo contributors SVG remains separate.

Source cleanup follows in trvny/trvny after this lands.

View guided diff Turn on auto-fix

Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>
@kanarek-companion

kanarek-companion Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

🐤 Kanarek · 🟣 merged

main ✅

Code is in the nest. The machinery may rest.

Repo root, Documentation · 2 files

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-13T18:47:21.159444Z
We couldn’t complete this review. Comment @codex review to try again.
ddd6670 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@coderabbitai

coderabbitai Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Explain Change →

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: c24f2f8e-0773-434e-aefa-71f71a225e04
Walkthrough

新增 .all-contributorsrc,配置 README.md 输出、展示选项及 29 位贡献者的信息。README.md 新增 All Contributors 规范说明和贡献者表格,列出贡献者及其贡献类别。

Merge Risk: Low · previous review ddd66

Regenerating the contributors table could silently repoint existing activity links to the wrong repository. Adding link-template overrides in the configuration before relying on regeneration avoids this. The impact is small and does not affect runtime behavior.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed 标题 "docs: move all-contributors roster to shared profile" 准确描述主要变更:将 All Contributors 贡献者名单迁移到共享 profile README,并新增 .all-contributorsrc 配置。标题简洁,类型前缀 docs 与变更性质(文档/配置)一致。
Description check Passed 描述说明将 31 项 All Contributors 名单从 trvny/trvny 迁移到共享 profile README,新增 .all-contributorsrc 作为规范配置,并保留现有表格中的活动链接。该描述与变更集直接相关。
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Finishing Touches

Simplify code

  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

石头刻名,名单长
七位一行,头像亮
配置记下贡献样
README 里都登场
洞穴开门,欢迎帮

@mergify

mergify Bot commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

This pull request is merged.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Comment thread .all-contributorsrc Resolved

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've found 2 issues

Prompt for AI Agents
Please address the comments from this code review:

## Individual Comments

### Comment 1
<location path="README.md" line_range="61" />
<code_context>
 - [Joke of the Day](https://djtech.net/humor/things_to_think_about.htm)
 <!--README_FEED:END-->
+
+## Contributors ✨
+
+This project follows the [all-contributors](https://github.com/all-contributors/all-contributors) specification. Contributions of any kind welcome!
</code_context>
<issue_to_address>
**Roster missing from profile**

When the old roster is removed from trvny/trvny or visitors open the account profile, gitHub renders the profile README from trvny/trvny, not trvny/.github, and the shared-README updater syncs only quote and feed blocks; the roster therefore stays off the profile and visitors do not see it.

Put and maintain the roster in trvny/trvny/README.md, or update the shared-README updater to sync it there before removing the old roster.

Also at `README.md:62-63`.
</issue_to_address>

### Comment 2
<location path="README.md" line_range="72" />
<code_context>
+      <td align="center" valign="top" width="14.29%"><a href="https://github.com/trvny"><img src="https://avatars.githubusercontent.com/u/120686325?u=f421c62e9d101e4f7ddac87061f82ccca98b2fe1&amp;v=4&amp;s=100" width="100px;" alt="trvny"/><br /><sub><b>trvny</b></sub></a><br /><a href="https://github.com/trvny/trvny/commits?author=trvny" title="Code">💻</a> <a href="https://github.com/trvny/trvny/commits?author=trvny" title="Documentation">📖</a> <a href="https://github.com/trvny" title="Maintenance">🚧</a></td>
</code_context>
<issue_to_address>
**Activity links point to wrong repository**

When the All Contributors bot or CLI regenerates the roster after an update, the generator uses projectOwner and projectName, with projectName set to .github, to rewrite the existing trvny/trvny activity URLs to trvny/.github; readers then see activity for the wrong repository.

Configure the generator to use trvny/trvny for activity links or preserve the existing URLs.
</issue_to_address>

Sourcery assessment

Approval pending. 2 findings to address first.

Blocking findings: README.md:61, README.md:72


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

Comment thread README.md Resolved
Comment thread README.md Resolved

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 2 files

Reply with feedback, questions, or to request a fix.

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread .all-contributorsrc Resolved
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>

@kanarek-companion kanarek-companion Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🐤 Kanarek 代码审查 · DeepSeek DeepSeek · deepseek-flash

新增的 .all-contributorsrc 与 README 中保留的活动链接存在目标仓库不一致,后续自动重新生成会覆盖已保留的链接。

Comment thread .all-contributorsrc Resolved
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>

@kanarek-companion kanarek-companion Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🐤 Kanarek 代码审查 · DeepSeek DeepSeek · deepseek-flash

新增的 .all-contributorsrc 中,贡献链接模板与声明的项目(trvny/.github)不一致,后续在本仓库新增的贡献者会被链接到 trvny/trvny 的活动记录。

Comment thread .all-contributorsrc Resolved
Co-authored-by: chatgpt-codex-connector[bot] <199175422+chatgpt-codex-connector[bot]@users.noreply.github.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

All reported issues were addressed across 1 file (changes from recent commits).

Reply with feedback, questions, or to request a fix.

View guided diff | Turn on auto-fix | Re-trigger cubic

Comment thread .all-contributorsrc Resolved
@trvny
trvny merged commit d19cf0a into main Nov 13, 2026
9 checks passed
@trvny
trvny deleted the docs/centralize-all-contributors-20261113 branch November 13, 2026 19:05
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