-
Notifications
You must be signed in to change notification settings - Fork 51
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
read sha as batch id #31
Conversation
@@ -23,6 +23,8 @@ abstract class ApplitoolsTest { | |||
Eyes().apply { | |||
componentsProvider = AndroidXComponentsProvider() | |||
configuration = Configuration().apply { | |||
appName = "Android Reference Assets" | |||
addProperty("platform", "android") | |||
batch = batchInfo | |||
apiKey = ApplitoolsConfig.APPLITOOLS_KEY |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you gonna try removing this in this PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
applitools support and i agreed, that since what we have works, it's not entirely worth it to figure out why we can't just pull from the environment.. i thought that maybe it had to do with those env vars not existing within the emulator, but not entirely sure.. regardless, i was a little skeptical anyways
41b15d3
to
184fa06
Compare
No description provided.