[PR #308] [CLOSED] docs: update CONTRIBUTING.md to reflect current review workflow #463

New issue

Closed

opened 2026-05-06 13:08:13 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-06 13:08:13 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cloudflare/vinext/pull/308
Author: @southpolesteve
Created: 3/7/2026
Status: ❌ Closed

Base: main ← Head: docs/update-contributing

📝 Commits (2)

d951af6 Skip deploy previews for fork PRs that lack Cloudflare secrets
2a42f98 docs: update CONTRIBUTING.md to reflect current review workflow and merge philosophy

📊 Changes

2 files changed (+9 additions, -8 deletions)

View changed files

📝 .github/workflows/deploy-examples.yml (+3 -3)
📝 CONTRIBUTING.md (+6 -5)

📄 Description

Summary

Updated the AI code review section to describe the actual workflow: maintainers with write access request BigBonk reviews, external contributors can't trigger it directly
Added explanation of the iterate-on-feedback process and that BigBonk suggestions aren't mandatory but are generally high quality
Added bias-towards-merging philosophy: this is a new project, we want to get PRs landed fast
Added note that maintainers may push changes or create new PRs, and will preserve original author credit
Removed stale reference to post-launch issue labels

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cloudflare/vinext/pull/308 **Author:** [@southpolesteve](https://github.com/southpolesteve) **Created:** 3/7/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `docs/update-contributing` --- ### 📝 Commits (2) - [`d951af6`](https://github.com/cloudflare/vinext/commit/d951af6a9bc4e890d8a74fc668b4d588ba923257) Skip deploy previews for fork PRs that lack Cloudflare secrets - [`2a42f98`](https://github.com/cloudflare/vinext/commit/2a42f98e1e203f719c5298b21ee1feaa18024d36) docs: update CONTRIBUTING.md to reflect current review workflow and merge philosophy ### 📊 Changes **2 files changed** (+9 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/deploy-examples.yml` (+3 -3) 📝 `CONTRIBUTING.md` (+6 -5) </details> ### 📄 Description ## Summary - Updated the AI code review section to describe the actual workflow: maintainers with write access request BigBonk reviews, external contributors can't trigger it directly - Added explanation of the iterate-on-feedback process and that BigBonk suggestions aren't mandatory but are generally high quality - Added bias-towards-merging philosophy: this is a new project, we want to get PRs landed fast - Added note that maintainers may push changes or create new PRs, and will preserve original author credit - Removed stale reference to `post-launch` issue labels --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>