Skip to content
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

Updated to use the output_directory from gym if provided #64

Open
asoneji opened this issue Feb 11, 2021 · 1 comment · May be fixed by #74
Open

Updated to use the output_directory from gym if provided #64

asoneji opened this issue Feb 11, 2021 · 1 comment · May be fixed by #74
Labels
good first issue Good for newcomers

Comments

@asoneji
Copy link

asoneji commented Feb 11, 2021

In our gym config, we have provided output_directory dir. When using IPASizeProvider we got the follwoing error:

[17:28:41]: $ ../Pods/SwiftInfo/bin/swiftinfo
[17:28:47]: ▸ * Extracting ipa_size
[17:28:47]: ▸ * **ipa_size:** .ipa not found! Attempted to find .ipa at: ../build/

I noticed is not using the output_directory dir from gym and instead, it is expecting the ipa to be inside the ./build folder.

@rockbruno
Copy link
Owner

Seems like we could to add a path parameter to IPASizeProvider. Should be an easy fix 👍

@rockbruno rockbruno added the good first issue Good for newcomers label Feb 11, 2021
@wacumov wacumov linked a pull request Oct 5, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants