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 default openBrowser function #89

Merged
merged 1 commit into from
May 4, 2023
Merged

Add default openBrowser function #89

merged 1 commit into from
May 4, 2023

Conversation

memreflect
Copy link
Contributor

All platforms other than Darwin, Linux, and Windows fail to build gup due to a missing openBrowser function.
This commit adds a default openBrowser function for those other systems to allow gup to build on other platforms like the BSDs, directing users to open the bug report URL in their preferred browser as intended.

All systems other than Darwin, Linux, and Windows fail to build gup due
to a missing openBrowser function.
The addition of a default openBrowser function for those other systems
allows gup to continue building on other platforms like the BSDs,
directing users to open the bug report URL in their preferred browser.
@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Merging #89 (1bf2a7b) into main (6ad20f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #89   +/-   ##
=======================================
  Coverage   82.84%   82.84%           
=======================================
  Files          15       15           
  Lines         688      688           
=======================================
  Hits          570      570           
  Misses         87       87           
  Partials       31       31           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nao1215
Copy link
Owner

nao1215 commented May 4, 2023

@memreflect
Thank you for your PR. I will merge this PR.
(Note that the gup command is officially only supported on Linux, Mac, and Windows.)

@nao1215 nao1215 merged commit 1d295d2 into nao1215:main May 4, 2023
@memreflect memreflect deleted the patch-openBrowser-default branch May 4, 2023 11:56
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