Repository navigation
Send external contributors to dev; extern-contrib is retired - #11
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E5dr7XTsCeoamqLSiT9rLp
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Deploying docs with
|
| Latest commit: |
2854b7b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://69896109.docs-42b.pages.dev |
| Branch Preview URL: | https://drop-extern-contrib.docs-42b.pages.dev |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2854b7ba41
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ``` | ||
|
|
||
| 提到 `master` 或 `dev` 的外部 PR 会被关掉。 | ||
| 再从你的 fork 向本仓库的 `dev` 提 PR。第一次提 PR 时，测试等 CI 要等维护者批准后才会运行，之后的 PR 会自动运行。版本号和更新记录由维护者和 CI 处理。 |
There was a problem hiding this comment.
For contributors whose first PR is closed without merging, or who open multiple PRs before any contribution is merged, later workflow runs still require maintainer approval. The first_time_contributors policy stops requiring approval only after the user has had a commit or PR merged, not merely after submitting their first PR, so “之后的 PR 会自动运行” gives those contributors incorrect expectations; describe the threshold as having a contribution merged instead. GitHub documents the policy semantics here.
Useful? React with 👍 / 👎.
1 participant
Documents the change in XMOJ-Script-dev/XMOJ-Script#1055. The
extern-contribbranch and its sync workflows are gone; fork PRs now targetdevdirectly.community/contributing.md: the branch table dropsextern-contrib, anddevtakes all PRs. Adds the fork →upstream/dev→ PR steps, notes that CI on a contributor's first PR waits for maintainer approval (the repo policy is nowfirst_time_contributors), and says version numbers are handled by maintainers and CI.🤖 Generated with Claude Code
https://claude.ai/code/session_01E5dr7XTsCeoamqLSiT9rLp