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

Print a message explaining how to quickly open links on macOS #7832

Merged
merged 1 commit into from
Jan 15, 2017

Conversation

0xced
Copy link
Contributor

@0xced 0xced commented Jan 11, 2017

Checklist

  • I've run bundle exec rspec from the root directory to see all new and existing tests pass
  • I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
  • I've read the Contribution Guidelines
  • I've updated the documentation if neccessary.

Description

A remake of orta/gh_inspector#14 for the custom inspector reporter.

Motivation and Context

Because ⌘ + double-click is much faster than

  1. double-click + drag
  2. ⌘ + C
  3. ⌘ + n × ⇥
  4. ⌘ + T
  5. ⌘ + V

Copy link
Member

@KrauseFx KrauseFx left a comment

Choose a reason for hiding this comment

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

Beautiful, thanks for this PR 👍

@KrauseFx
Copy link
Member

Not sure what's up with Circle for this PR, I guess it's due to the GitHub downtime?

@hjanuschka
Copy link
Collaborator

fantastic branch name 👍 how do we re-trigger ci? (can you push a commit?)


def print_open_link_hint(newline = false)
puts "" if newline
puts "🔗 You can ⌘ + double-click on links to open them directly in your browser." if /darwin/ =~ RUBY_PLATFORM
Copy link
Contributor

Choose a reason for hiding this comment

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

If we need another change to re-trigger CI... can we replace the end check with Helper.mac?

Because ⌘ + double-click is much faster than
1. double-click + drag
2. ⌘ + C
3. ⌘ + n × ⇥
4. ⌘ + T
5. ⌘ + V
6. ⏎

A remake of orta/gh_inspector#14 for the custom inspector reporter.
@KrauseFx
Copy link
Member

Beautiful, thanks @0xced 👍

@asfalcone asfalcone mentioned this pull request Jan 17, 2017
@fastlane fastlane locked and limited conversation to collaborators Apr 15, 2017
@0xced 0xced deleted the 🔗🌍🚀 branch April 26, 2017 08:31
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

4 participants