Skip to content

Conversation

@Jakeii
Copy link
Member

@Jakeii Jakeii commented Nov 20, 2025

What does this change?

Create a new directory ab-testing/config for the ab testing config code, this in now a workspace directory. and ab-testing is no longer a workspace.

Why?

There will be some more elements joining the ab-testing directory - lambda and cdk code. This is to avoid workspaces within workspaces.

@Jakeii Jakeii requested review from a team as code owners November 20, 2025 14:35
@github-actions
Copy link

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@github-actions
Copy link

github-actions bot commented Nov 20, 2025

@github-actions
Copy link

github-actions bot commented Nov 20, 2025

@Jakeii Jakeii force-pushed the jlk/ab-testing-config-workspace branch from f5bf708 to 7f351c7 Compare November 20, 2025 14:38
@Jakeii Jakeii added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Nov 20, 2025
Copy link
Contributor

@on-ye on-ye left a comment

Choose a reason for hiding this comment

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

I think the separation of workspaces through the pnpm-workspace.yaml file isn't as intuitive as looking at the packages directory, but I guess that's an organisational pattern.

Still, I think this PR is a good step to increased clarity, and potentially moving ab testing to it's own repo

@Jakeii Jakeii added the run_chromatic Runs chromatic when label is applied label Nov 21, 2025
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Nov 21, 2025
@Jakeii Jakeii merged commit 1c30115 into main Nov 21, 2025
42 checks passed
@Jakeii Jakeii deleted the jlk/ab-testing-config-workspace branch November 21, 2025 16:26
@gu-prout
Copy link

gu-prout bot commented Nov 21, 2025

Seen on PROD (merged by @Jakeii 8 minutes and 36 seconds ago) Please check your changes!

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

Labels

dotcom-rendering maintenance Departmental tracking: maintenance work, not a fix or a feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants