Skip to content

docs(ab-testing): rewrite onboarding guide#28723

Merged
bfullam merged 1 commit into
mainfrom
swaps-improve-ab-test-docs
Apr 13, 2026
Merged

docs(ab-testing): rewrite onboarding guide#28723
bfullam merged 1 commit into
mainfrom
swaps-improve-ab-test-docs

Conversation

@bfullam

@bfullam bfullam commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Description

This updates the A/B testing documentation to make the implementation path much easier for a new engineer to follow.

The guide now starts with a quickstart and definition of done, consolidates the implementation guidance into one end-to-end example, and moves agent-specific workflow details to an appendix so they do not interrupt the main onboarding flow.

Changelog

CHANGELOG entry: null

Related issues

Fixes:

Manual testing steps

N/A

Screenshots/Recordings

Before

N/A

After

N/A

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@github-actions

Copy link
Copy Markdown
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-swaps-and-bridge Swaps and Bridge team label Apr 13, 2026
@bfullam bfullam changed the title Update A/B testing docs for faster onboarding docs(ab-testing): rewrite onboarding guide Apr 13, 2026
@github-actions github-actions Bot added the risk-low Low testing needed · Low bug introduction risk label Apr 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: None (no tests recommended)
  • Selected Performance tags: None (no tests recommended)
  • Risk Level: low
  • AI Confidence: 99%
click to see 🤖 AI reasoning details

E2E Test Selection:
The only changed file is docs/ab-testing.md, which is a pure documentation file. The diff shows a restructuring and reformatting of the A/B testing guide — adding a Quickstart section, FAQ, clearer examples, and reorganizing existing content. There are zero code changes, no modifications to any app logic, controllers, hooks, analytics utilities, test infrastructure, or CI configuration. This change has no impact on any user-facing functionality or E2E test flows, so no E2E tests need to run.

Performance Test Selection:
Pure documentation change with no code modifications. No performance impact whatsoever.

View GitHub Actions results

@bfullam bfullam enabled auto-merge April 13, 2026 10:51
@sonarqubecloud

Copy link
Copy Markdown

@bfullam bfullam added this pull request to the merge queue Apr 13, 2026
Merged via the queue into main with commit 3d1b381 Apr 13, 2026
80 of 86 checks passed
@bfullam bfullam deleted the swaps-improve-ab-test-docs branch April 13, 2026 11:19
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 13, 2026
@metamaskbot metamaskbot added the release-7.74.0 Issue or pull request that will be included in release 7.74.0 label Apr 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.74.0 Issue or pull request that will be included in release 7.74.0 risk-low Low testing needed · Low bug introduction risk size-L team-swaps-and-bridge Swaps and Bridge team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants