Skip to content

docs: Consolidate PR collaboration guide to Flagsmith/AGENTS.md#7646

Open
emyller wants to merge 4 commits into
mainfrom
docs/point-to-agents-md
Open

docs: Consolidate PR collaboration guide to Flagsmith/AGENTS.md#7646
emyller wants to merge 4 commits into
mainfrom
docs/point-to-agents-md

Conversation

@emyller
Copy link
Copy Markdown
Contributor

@emyller emyller commented May 29, 2026

Our pull-request playbook now lives at https://github.com/Flagsmith/AGENTS.md so it can be shared across repositories and consumed by review agents from a single source. This PR consolidates Flagsmith's own references against that new home.

Changes

  • CONTRIBUTING.md points contributors at the consolidated playbook in Flagsmith/AGENTS.md and drops the steps now covered there (draft PRs, addressing feedback).
  • .gemini/styleguide.md directs Gemini Code Assist at the same playbook. The styleguide is only consulted once .gemini/config.yaml is in place — should land in chore: Add configuration files for Gemini PR reviews #7577.

Note

I'm not sure if Gemini will follow the link, or only read inline instructions. We'll see. IMO guidelines aren't much relevant to Gemini reviews anyway.

Closes Flagsmith/AGENTS.md#1

Review effort: 1/5

@emyller emyller requested a review from a team as a code owner May 29, 2026 18:38
@emyller emyller requested review from gagantrivedi and removed request for a team May 29, 2026 18:38
@vercel
Copy link
Copy Markdown

vercel Bot commented May 29, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored Preview May 29, 2026 6:42pm
flagsmith-frontend-preview Ignored Ignored Preview May 29, 2026 6:42pm
flagsmith-frontend-staging Ignored Ignored Preview May 29, 2026 6:42pm

Request Review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the CONTRIBUTING.md file by removing redundant instructions on draft PRs and addressing feedback, and adding a reference to the PR_COLLABORATION.md guide. The reviewer suggested a phrasing improvement to make the collaboration guide sound more encouraging and less optional for external contributors.

Comment thread CONTRIBUTING.md Outdated
@github-actions github-actions Bot added the docs Documentation updates label May 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Docker builds report

Image Build Status Security report
ghcr.io/flagsmith/flagsmith-api-test:pr-7646 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-e2e:pr-7646 Finished ✅ Skipped
ghcr.io/flagsmith/flagsmith-frontend:pr-7646 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-api:pr-7646 Finished ✅ Results
ghcr.io/flagsmith/flagsmith:pr-7646 Finished ✅ Results
ghcr.io/flagsmith/flagsmith-private-cloud:pr-7646 Finished ✅ Results

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@github-actions github-actions Bot added docs Documentation updates and removed docs Documentation updates labels May 29, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  37.8 seconds
commit  47870f5
info  🔄 Run: #17117 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  40.4 seconds
commit  47870f5
info  🔄 Run: #17117 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  3 passed

Details

stats  3 tests across 3 suites
duration  33.1 seconds
commit  47870f5
info  🔄 Run: #17117 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  1 minute, 6 seconds
commit  47870f5
info  🔄 Run: #17117 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  39 seconds
commit  e73d634
info  🔄 Run: #17118 (attempt 1)

Playwright Test Results (oss - depot-ubuntu-latest-arm-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  42.4 seconds
commit  e73d634
info  🔄 Run: #17118 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-16)

passed  2 passed

Details

stats  2 tests across 2 suites
duration  32.1 seconds
commit  e73d634
info  🔄 Run: #17118 (attempt 1)

Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)

passed  1 passed

Details

stats  1 test across 1 suite
duration  39.4 seconds
commit  e73d634
info  🔄 Run: #17118 (attempt 1)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

Visual Regression

19 screenshots compared. See report for details.
View full report

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consolidate team PR review/collaboration guide to docs

1 participant