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

[deliver] fix double screenshots directory on deliver setup #13201

Conversation

joshdholtz
Copy link
Member

Fixes #13194

Problem

The full screenshots path was getting generated here -

screenshots_path = options[:screenshots_path] || File.join(deliver_path, 'screenshots')

But then screenshots was getting appended to the path again here -

path = File.join(deliver_path, 'screenshots')

Solution

Removed the second screenshots from getting appended in the download_screenshots method

@KrauseFx
Copy link
Member

This fixes the newly created screenshots folder, however I have the feeling something else is going wrong here, e.g. #13195.

I'm worried something broke recently (e.g. #13176)

@joshdholtz
Copy link
Member Author

@KrauseFx Aight, I will look into those too 👌

@joshdholtz joshdholtz merged commit 08d7ebd into master Aug 27, 2018
@joshdholtz joshdholtz deleted the joshdholtz-fix-double-screenshots-directory-on-deliver-setup branch August 27, 2018 12:32
@fastlane-bot
Copy link

Hey @joshdholtz 👋

Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉
The code change now lives in the master branch, however it wasn't released to RubyGems yet.
We usually ship about once a week, and your PR will be included in the next one.

Please let us know if this change requires an immediate release by adding a comment here 👍
We'll notify you once we shipped a new release with your changes 🚀

@fastlane-bot
Copy link

Congratulations! 🎉 This was released as part of fastlane 2.103.0 🚀

janpio pushed a commit to janpio/fastlane that referenced this pull request Sep 11, 2018
@fastlane fastlane locked and limited conversation to collaborators Nov 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants