Skip to content

chore(warehouse_sources): promote Brevo to GA - #95864

Merged
trunk-io[bot] merged 1 commit into
masterfrom
posthog/promote-brevo-ga
Sep 7, 2026
Merged

chore(warehouse_sources): promote Brevo to GA#95864
trunk-io[bot] merged 1 commit into
masterfrom
posthog/promote-brevo-ga

Conversation

@Gilbert09

Copy link
Copy Markdown
Member

Problem

Anyone browsing the data warehouse source catalog sees Brevo labelled Beta, which reads as "not ready for production yet". It is ready — the connector has been live for over three months with a clean import record, so the label is now understating it and gives teams a reason to hesitate before connecting.

Changes

  • Brevo now shows as GA in the new-source wizard and on its docs page, instead of Beta.
  • Mechanical: one field on BrevoSource.get_source_configreleaseStatus goes from ReleaseStatus.BETA to ReleaseStatus.GA.

Brevo carries no unreleasedSource flag and no featureFlag gate, so nothing else had to be unwound. It was already visible and connectable to everyone; only the label changes.

The promotion bar it clears:

  • Adoption maturity: live 3.1 months (bar is 100+ teams or 3 months).
  • Import error rate over the last 7 days: 0.0% (bar is under 2.5%).

How did you test this code?

No new tests. releaseStatus is a declared label with no branching on it, so a test asserting the new value would only restate the diff.

No automated tests were run for this change, and no manual testing was done — the connector's request, schema, and sync paths are untouched, so the existing Brevo suites cover the same ground they did before. CI reports the results.

Automatic notifications

  • Publish to changelog?

Docs update

The posthog.com Brevo doc renders its release status from the public_source_configs API, so it picks the change up automatically.

🤖 Agent context

Autonomy: Fully autonomous

Tooling: Claude Code (Opus 5), via the gh CLI and repo file tools. Skills invoked: /implementing-warehouse-sources (for the release-status conventions and the ReleaseStatus enum rule), /writing-pr-descriptions.

Decisions along the way:

  • Followed the enum convention rather than dropping releaseStatus entirely. Omitting the field also means GA, but the explicit ReleaseStatus.GA matches what every other GA source in the tree does.
  • Left gating alone. The skill only calls for removing unreleasedSource / featureFlag when they are present, and Brevo has neither.

No duplicate: searched open PRs for "brevo", "releaseStatus", and "promote", and listed the maintainer's open PRs. The nearest match, #95842, promotes ActiveCampaign and Mailjet and touches neither Brevo file. Nothing else addresses this.

Public artifact: the diff is one enum value. The adoption and error-rate figures quoted above are this connector's own aggregate health metrics, not customer data, and no session material reached the branch.


Created with PostHog Desktop

Brevo has met the GA bar for adoption maturity and import reliability, so its release status moves from beta to GA. No connector behavior, schema, or sync logic changes.

Generated-By: PostHog Desktop
Task-Id: 17a86e58-8d01-4e51-bb02-168ad73783df
@Gilbert09 Gilbert09 self-assigned this Sep 7, 2026
Copilot AI lite review requested due to automatic review settings September 7, 2026 05:03
@trunk-io

trunk-io Bot commented Sep 7, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@pr-assigner-resolver-posthog
pr-assigner-resolver-posthog Bot requested a review from a team September 7, 2026 05:04
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

🤖 CI report

⚠️ Trunk lane — backend Python lane

This PR is assigned to the backend Python lane. It runs backend Python tests and may merge in parallel with PRs in other lanes.

@Gilbert09 Gilbert09 added the stamphog Request AI approval (no full review) label Sep 7, 2026 — with PostHog

@stamphog stamphog 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.

Approved.

Single-line release-status label flip on a source the author owns and has strong familiarity with; no behavioral or risky-territory impact.

  • Author wrote 100% of the modified lines and has 2 merged PRs in these paths (familiarity STRONG).
  • copilot-pull-request-reviewer[bot] reviewed the current head.
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 2L, 1F substantive — within ceiling
tier T1-agent / T1a-trivial (2L, 1F, single-area, chore)
stamphog 2.0.0b4 .stamphog/policy.yml @ f85983e · reviewed head f85983e

@trunk-io

trunk-io Bot commented Sep 7, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@talyn-app

talyn-app Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

/trunk merge

@trunk-io
trunk-io Bot merged commit 4502ec0 into master Sep 7, 2026
242 checks passed
@trunk-io
trunk-io Bot deleted the posthog/promote-brevo-ga branch September 7, 2026 12:12
@deployment-status-posthog

deployment-status-posthog Bot commented Sep 7, 2026

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-09-07 12:32 UTC Run
prod-us ✅ Deployed 2026-09-07 12:42 UTC Run
prod-eu ✅ Deployed 2026-09-07 12:45 UTC Run

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

Labels

stamphog Request AI approval (no full review)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants