Skip to content

Clarify review requirements for forked applications - #625

Merged
ruibaby merged 1 commit into
mainfrom
docs/fork-app-review-guidelines
Sep 7, 2026
Merged

Clarify review requirements for forked applications#625
ruibaby merged 1 commit into
mainfrom
docs/fork-app-review-guidelines

Conversation

@ruibaby

@ruibaby ruibaby commented Sep 7, 2026

Copy link
Copy Markdown
Member

Clarify when forked plugins and themes qualify for an independent marketplace listing, reducing minor variants that increase users' selection burden.

  • Require upstream contribution attempts, source disclosure, verifiable differences, and a maintenance plan.
  • Reject minor cosmetic changes, personal preferences, and AI rewrites without sufficient independent value.
  • Require evidence of discontinued upstream maintenance and completed maintenance work for successor applications.
  • Distinguish discontinued projects from repository migrations and inactivity.
  • Update the submission checklist and guideline version/effective date to 2026-09-07.

Validation:

  • pnpm check:ci
  • pnpm build
  • git diff --check
  • Verified page rendering and navigation to the new sections in the local development preview.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 7, 2026

Copy link
Copy Markdown

Codex Review Summary

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

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-07T02:45:09.756816Z 6457e4e 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.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs 6457e4e Commit Preview URL

Branch Preview URL
Sep 07 2026, 02:44 AM

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6457e4e777

ℹ️ 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".

- 涉及用户数据、站点内容、订单、许可证、日志或第三方服务时,已提供清晰说明
- 已按免费应用提交;如依赖第三方付费服务、外部授权或功能限制,已准确披露
- 开源许可证、第三方资源、商标、图片、字体、图标和模板授权清晰
- 如应用基于已有插件或主题派生,已披露上游来源,并提供上游贡献记录、实质差异说明和持续维护计划;以接续维护为由申请上架的,还需提供上游明确停止维护的证据

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Preserve the exception for missing contribution records

For a derived application with no upstream contribution history, this checklist says a contribution record is required, while §4.2.1 explicitly permits an honest explanation when no record exists (and §4.2.2 allows the same when contribution channels are unavailable). Applicants following the pre-submission checklist could therefore conclude that they are ineligible despite satisfying the detailed rules; require a contribution record or an explanation for its absence here.

Useful? React with 👍 / 👎.

@JohnNiang JohnNiang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@ruibaby
ruibaby merged commit 42df80b into main Sep 7, 2026
2 checks passed
@ruibaby
ruibaby deleted the docs/fork-app-review-guidelines branch September 7, 2026 03:12
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.

2 participants