Skip to content

Conversation

aanorbel
Copy link
Member

Closes #534

@aanorbel aanorbel marked this pull request as draft March 10, 2025 18:00
@aanorbel aanorbel requested a review from sdsantos March 13, 2025 10:37
@aanorbel aanorbel marked this pull request as ready for review March 13, 2025 10:37
Copy link
Contributor

@sdsantos sdsantos left a comment

Choose a reason for hiding this comment

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

I'm guessing this is a first pass, since we don't have the screenshots code working yet, right?

scheme: "#{scheme}UITests",
test_target_name: scheme,
erase_simulator: true,
# languages: locales,
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we need this?

Copy link
Member Author

Choose a reason for hiding this comment

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

We do.

Comment on lines +16 to +20
ooni/ios/*/*.png
ooni/ios/*.html

dw/ios/*/*.png
dw/ios/*.html
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't we want to eventually upload the screenshots into git?

Copy link
Member Author

Choose a reason for hiding this comment

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

We do. But these are general rules. we generate more screenshots that we actually need to upload to git.

@aanorbel aanorbel merged commit 4147f54 into main Mar 18, 2025
3 checks passed
@aanorbel aanorbel deleted the issues/534 branch March 18, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CD: Fastlane action to generate iOS screenshots

2 participants