Closed jmuzina closed 2 months ago
We should update the Percy prepare workflow so that the latest changes from main are included in the testing artifacts.
main
The current Percy workflow checks out the pull request source here. We should update this to check out a merge commit or to rebase against main.
If any merge/rebase conflicts are found, the workflow should immediately fail, prompting contributor/maintainer to rebase/merge themselves.
This will reduce cases where perceived visual regressions are seen in tests, only because the PR was not merged or rebased against main.
Thank you for reporting us your feedback!
The internal ticket has been created: https://warthogs.atlassian.net/browse/WD-13965.
This message was autogenerated
We should update the Percy prepare workflow so that the latest changes from
main
are included in the testing artifacts.The current Percy workflow checks out the pull request source here. We should update this to check out a merge commit or to rebase against main.
If any merge/rebase conflicts are found, the workflow should immediately fail, prompting contributor/maintainer to rebase/merge themselves.
This will reduce cases where perceived visual regressions are seen in tests, only because the PR was not merged or rebased against
main
.