Skip to content

test(tree): move all snapshots to under a directory that only contains snapshots#25780

Merged
yann-achard-MS merged 2 commits intomicrosoft:mainfrom
yann-achard-MS:move-all-snapshots
Oct 30, 2025
Merged

test(tree): move all snapshots to under a directory that only contains snapshots#25780
yann-achard-MS merged 2 commits intomicrosoft:mainfrom
yann-achard-MS:move-all-snapshots

Conversation

@yann-achard-MS
Copy link
Contributor

@yann-achard-MS yann-achard-MS commented Oct 30, 2025

Description

This PR moves all existing snapshots under from src/test/snapshots/ to src/test/snapshots/output.
This ensures that all snapshots are under a directory that only contains snapshots. A follow-up PR will leverage that to change how snapshots are regenerated.

Breaking Changes

None

@github-actions github-actions bot added area: dds Issues related to distributed data structures area: dds: tree base: main PRs targeted against main branch and removed area: dds Issues related to distributed data structures area: dds: tree labels Oct 30, 2025
@yann-achard-MS yann-achard-MS marked this pull request as ready for review October 30, 2025 23:16
@yann-achard-MS yann-achard-MS requested a review from a team as a code owner October 30, 2025 23:16
Copilot AI review requested due to automatic review settings October 30, 2025 23:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

@yann-achard-MS yann-achard-MS enabled auto-merge (squash) October 30, 2025 23:48
@yann-achard-MS yann-achard-MS merged commit 533b327 into microsoft:main Oct 30, 2025
34 of 35 checks passed
@yann-achard-MS yann-achard-MS deleted the move-all-snapshots branch November 3, 2025 21:49
anthony-murphy-agent pushed a commit to anthony-murphy-agent/FluidFramework that referenced this pull request Jan 14, 2026
…s snapshots (microsoft#25780)

## Description

This PR moves all existing snapshots under from `src/test/snapshots/` to
`src/test/snapshots/output`.
This ensures that all snapshots are under a directory that only contains
snapshots. A follow-up PR will leverage that to change how snapshots are
regenerated.

## Breaking Changes

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

Labels

base: main PRs targeted against main branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants