-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix screenshot recording in CI #3607
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3607 +/- ##
========================================
Coverage 82.68% 82.68%
========================================
Files 1731 1731
Lines 41008 41008
Branches 4973 4973
========================================
Hits 33906 33906
Misses 5342 5342
Partials 1760 1760 ☔ View full report in Codecov by Sentry. |
The tests passed, so I guess deleting those screenshots was the right thing for the flow to do. |
Yes, the Preview has been removed here: https://github.com/element-hq/element-x-android/pull/3597/files#diff-50bc7833b8f4ad56f8bf8ac0ce9b0b5e055250aeb602115fa22666b858a4493f but since the screenshots has not been recorded on this PR, the screenshot files have not been removed. |
Content
Fix recording screenshots flow.
Motivation and context
The
--no-configuration-cache
was added in the wrong place, it's a gradle argument, not a jvm option.Screenshots / GIFs
It somehow deleted some screenshots, which scares me a bit.
Checklist