Skip to content

Conversation

@jsuarezruiz
Copy link
Contributor

Description of Change

Currently, we publish the snapshots diffs (or new ones, those that don't exist) under the drop folder. For those contributing, can access to the artifacts and download the entire drop folder but it's almost 2GB of data between libraries, logs, images and more.

image

This PR adds changes to create folders directly in the root of the artifacts if necessary:

  • uitest-snapshot-results-ios
  • uitest-snapshot-results-android
  • uitest-snapshot-results-windows
  • uitest-snapshot-results-mac

This way, can download only the necessary images from the desired platform.

Copilot AI review requested due to automatic review settings February 20, 2025 12:48
@jsuarezruiz jsuarezruiz requested a review from a team as a code owner February 20, 2025 12:48
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 reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

@rmarinho
Copy link
Member

/azp run MAUI-UITests-public

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

Copy link
Member

@rmarinho rmarinho left a comment

Choose a reason for hiding this comment

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

Pipeline is failing to run

Screenshot 2025-04-15 at 12 56 00

@MartyIX
Copy link
Contributor

MartyIX commented Apr 15, 2025

Pipeline is failing to run

Your link appears to be wrong.

@PureWeen PureWeen added this to the .NET 9 SR10 milestone Jun 30, 2025
@PureWeen PureWeen moved this from Todo to Ready To Review in MAUI SDK Ongoing Jun 30, 2025
@PureWeen PureWeen modified the milestones: .NET 9 SR10, .NET 9 SR12 Aug 4, 2025
@jfversluis jfversluis force-pushed the publish-snapshots-artifacts branch from 9bb270a to 91f43a2 Compare August 11, 2025 14:24
Copy link
Member

@PureWeen PureWeen left a comment

Choose a reason for hiding this comment

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

  • just adding a request change on here so we hold on this until branching for SR10

@github-project-automation github-project-automation bot moved this from Ready To Review to Changes Requested in MAUI SDK Ongoing Aug 11, 2025
@jfversluis jfversluis requested a review from PureWeen August 12, 2025 14:06
@jfversluis jfversluis merged commit f4bf2fb into main Aug 12, 2025
130 checks passed
@jfversluis jfversluis deleted the publish-snapshots-artifacts branch August 12, 2025 14:07
@github-project-automation github-project-automation bot moved this from Changes Requested to Done in MAUI SDK Ongoing Aug 12, 2025
rmarinho pushed a commit that referenced this pull request Aug 13, 2025
…der (#27928)

* Publish new snapshots and diffs to a specific artifacts folder

* Fix mistake

* Updated test to generate new snapshots (and verify the changes)

* More changes

* Fix identation issues

* Removed dependencies usage

* More fixes

* Revert changes
@github-actions github-actions bot locked and limited conversation to collaborators Sep 12, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants