Skip to content

Fix the ask to open in browser dialog on Mac.#143

Merged
codenamev merged 1 commit intoreenhanced:masterfrom
verypossible:ask-to-open-fix
Oct 27, 2015
Merged

Fix the ask to open in browser dialog on Mac.#143
codenamev merged 1 commit intoreenhanced:masterfrom
verypossible:ask-to-open-fix

Conversation

@squaresurf
Copy link
Copy Markdown
Contributor

Why:

  • GitReflow::Sandbox#ask_to_open_in_browser method currently only works
    for linux.

This change addresses the need by:

  • Add OS.mac? to the initial if statement guarding other platforms.

Comment thread lib/git_reflow/sandbox.rb Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think there's an OS.unix? we should use here.

Why:

* GitReflow::Sandbox#ask_to_open_in_browser method currently only works
  for linux.

This change addresses the need by:

* Add OS.mac? to the initial if statement guarding other platforms.
@squaresurf
Copy link
Copy Markdown
Contributor Author

@codenamev I've fixed it to use OS.unix? 😄

@codenamev
Copy link
Copy Markdown
Collaborator

lgtm

@codenamev codenamev merged commit e0f5e22 into reenhanced:master Oct 27, 2015
codenamev pushed a commit that referenced this pull request Oct 27, 2015
Why:

GitReflow::Sandbox#ask_to_open_in_browser method currently only works
for linux.
This change addresses the need by:

Add OS.mac? to the initial if statement guarding other platforms.

Close #143
LGTM given by: @codenamev
@squaresurf squaresurf deleted the ask-to-open-fix branch October 27, 2015 13:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants