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

Add missing device names for iPhone7 and iPhone7Plus #6741

Merged
merged 1 commit into from
Oct 25, 2016

Conversation

mpirri
Copy link
Contributor

@mpirri mpirri commented Oct 25, 2016

Fix issue #6376 - Screenshots on iPhone 7 and iPhone 7 Plus aren't working.

Thanks for contributing to fastlane! Before you submit your pull request, please make sure to check the following boxes:

  • Run bundle exec rspec from the subdirectory of each tool you modified. Alternatively, run rake test_all from the root directory.
  • Run bundle exec rubocop -a to ensure the code style is valid
  • Read the Contribution Guidelines
  • We currently don't accept new actions, please publish a plugin instead, more information in Plugins.md

Before submitting a pull request, we appreciate if you create an issue first to discuss the change 👍

Copy link
Contributor

@ohayon ohayon left a comment

Choose a reason for hiding this comment

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

This looks good for sure. Not sure why tests are failing, but seems legit!

@KrauseFx
Copy link
Member

@ohwutup you approved the PR fixing the tests #6724 😉
@mpirri can you rebase from master to fix the tests?

@mpirri mpirri force-pushed the feature/fix-iPhone7-snapshot branch from 9606fb7 to 243cb06 Compare October 25, 2016 21:24
@mpirri mpirri merged commit 50ce2d9 into master Oct 25, 2016
@mpirri mpirri deleted the feature/fix-iPhone7-snapshot branch October 25, 2016 21:39
mpirri pushed a commit that referenced this pull request Oct 27, 2016
* Add missing device names for iPhone7 and iPhone7Plus (#6741)
* Update internal dependencies (#6682)
* Update fastlane tagline (#6645)
* Update fastlane_core dependencies (#6615)
* Update internal dependencies (#6599)
* Update Snapshot url in runner file (#6609)
* Update rubocop and update styling rules (#6573)
* Fix typo (#6596)
* Update SnapshotHelper.swift (#6581)
* Show Xcode path in summary table (#6472)
* Update xcpretty dependency (#6510)
* Losen up dependencies of subtools (#6461)
* Add link to new fastlane docs (#6447)
* Replace `snapshot` with _snapshot_ (#6448)
* Fix information about how to access the current language (#6449)
@mpirri mpirri mentioned this pull request Oct 27, 2016
4 tasks
mpirri added a commit that referenced this pull request Oct 27, 2016
* Add missing device names for iPhone7 and iPhone7Plus (#6741)
* Update internal dependencies (#6682)
* Update fastlane tagline (#6645)
* Update fastlane_core dependencies (#6615)
* Update internal dependencies (#6599)
* Update Snapshot url in runner file (#6609)
* Update rubocop and update styling rules (#6573)
* Fix typo (#6596)
* Update SnapshotHelper.swift (#6581)
* Show Xcode path in summary table (#6472)
* Update xcpretty dependency (#6510)
* Losen up dependencies of subtools (#6461)
* Add link to new fastlane docs (#6447)
* Replace `snapshot` with _snapshot_ (#6448)
* Fix information about how to access the current language (#6449)
nareshsvs pushed a commit to nareshsvs/fastlane that referenced this pull request Oct 28, 2016
* master_original: (286 commits)
  [fastlane] Don’t ask for clipboard on non-interactive shell (fastlane#6782)
  [spaceship] Fix access control of `mac?` method in upload_file (fastlane#6783)
  [scan] Handle single scheme Xcode projects (fastlane#6637)
  spaceship doesn't have access to FastlaneCore::Helper (fastlane#6778)
  [snapshot] Version Bump (fastlane#6777)
  Update "spaceship in use" section (fastlane#6763)
  Add instructions for `fastlane env` in fastlane issue reporter (fastlane#6775)
  [spaceship] Add instructions on how to use 2factor on non-interactive shells (fastlane#6764)
  Improve grammar for gym message, and provide sample usage (fastlane#6776)
  [spaceship] Skips removing of alpha channel on non-Mac hosts (fastlane#6767)
  make sure that languages are available on iTC if you have localized screenshots (fastlane#6760)
  make use of the rollout option when promoting from a track to the sta… (fastlane#6502)
  [fastlane] Remove extra * in env printer (fastlane#6745)
  Add missing device names for iPhone7 and iPhone7Plus (fastlane#6741)
  [deliver] Version bump (fastlane#6744)
  [fastlane] Improve code style of xcversion (fastlane#6740)
  [fastlane] Version bump (fastlane#6723)
  Update internal dependencies (fastlane#6724)
  Fix spelling mistake (fastlane#6713)
  Add `SPACESHIP_TIMEOUT` environment variable to change the default timeout (fastlane#6712)
  ...

# Conflicts:
#	match/lib/match/git_helper.rb
#	match/lib/match/runner.rb
#	match/lib/match/version.rb
@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.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants