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

Fixed Scan destination parameter #5744

Merged
merged 1 commit into from
Aug 13, 2016
Merged

Fixed Scan destination parameter #5744

merged 1 commit into from
Aug 13, 2016

Conversation

bartoszj
Copy link
Contributor

Bug was described in task #5731.

Destination parameter, provided in the Fastfile, was always removed by line:

Scan.config[:destination] = nil # we have to do this, as otherwise a warning is shown to the user to not set this value

My suggestion is to save values[:destination] and restore it just before running Scan action instead of setting it always to nil.

@KrauseFx KrauseFx merged commit 3dfdc7f into fastlane:master Aug 13, 2016
@KrauseFx
Copy link
Member

Perfect, thank you @bartoszj - and sorry for introducing the bug

@bartoszj bartoszj deleted the bugfix/scan_destination branch August 14, 2016 08:51
@vpolouchkine vpolouchkine mentioned this pull request Aug 15, 2016
2 tasks
asfalcone pushed a commit that referenced this pull request Aug 18, 2016
[10:47:00]: Changes since release 1.100.0:
* Reduce output when running fastlane -v (#5787)
* Device specific warnings in S3 HTML template (#5775)
* Add a flag dry_run_email flag to lane generated by init beta (#5772)
* typo: sepcific -> specific (#5769)
* Choose non interactive (#5738)
* Add Circle.md (#5750)
* Fixed Scan `destination` parameter (#5744)
* [fastlane] latest_testflight_build_number use itc_team_id (#5725)
* Do not fail if no version exists on iTunes Connect (#5590)
* Add groups support to init beta (#5723)
* Skip GitHub issues by default for user_error! (#5622)
* [fastlane] Update fastlane_core dependency (#5721)
* [fastlane] Fix spelling mistake (#5724)
* Add ignore option to commit_version_bump action. (#5541)
* Fix #5571, check the absence of 'manifest' in export_options_plist (#5586)
* [fastlane] Update metadata of git_tag_exists (#5714)
* Disable the rubocop rules for MethodLength and AbcSize (#5715)
* Add option to log output for `import_certificate` per #1314 (#5321)
* Make commander great again (#5612)
* (feat/actions): Add new action 'check if the given git tag exists in the repo or not' (#5555)
* Fix documentation about zip action (#5625)
@asfalcone asfalcone mentioned this pull request Aug 18, 2016
asfalcone pushed a commit that referenced this pull request Aug 18, 2016
[10:47:00]: Changes since release 1.100.0:
* Reduce output when running fastlane -v (#5787)
* Device specific warnings in S3 HTML template (#5775)
* Add a flag dry_run_email flag to lane generated by init beta (#5772)
* typo: sepcific -> specific (#5769)
* Choose non interactive (#5738)
* Add Circle.md (#5750)
* Fixed Scan `destination` parameter (#5744)
* [fastlane] latest_testflight_build_number use itc_team_id (#5725)
* Do not fail if no version exists on iTunes Connect (#5590)
* Add groups support to init beta (#5723)
* Skip GitHub issues by default for user_error! (#5622)
* [fastlane] Update fastlane_core dependency (#5721)
* [fastlane] Fix spelling mistake (#5724)
* Add ignore option to commit_version_bump action. (#5541)
* Fix #5571, check the absence of 'manifest' in export_options_plist (#5586)
* [fastlane] Update metadata of git_tag_exists (#5714)
* Disable the rubocop rules for MethodLength and AbcSize (#5715)
* Add option to log output for `import_certificate` per #1314 (#5321)
* Make commander great again (#5612)
* (feat/actions): Add new action 'check if the given git tag exists in the repo or not' (#5555)
* Fix documentation about zip action (#5625)
@fastlane fastlane locked and limited conversation to collaborators Feb 4, 2017
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

2 participants