Skip to content

fix: update file opening command for cross-platform compatibility#22

Merged
LeperGnome merged 1 commit intoLeperGnome:mainfrom
jarpex:main
Sep 14, 2025
Merged

fix: update file opening command for cross-platform compatibility#22
LeperGnome merged 1 commit intoLeperGnome:mainfrom
jarpex:main

Conversation

@jarpex
Copy link
Copy Markdown
Contributor

@jarpex jarpex commented Sep 8, 2025

Implemented macOS-specific support for opening files by replacing xdg-open with the native open command. This ensures files are opened correctly on macOS systems, as xdg-open is Linux-specific and not available by default on macOS.

@LeperGnome
Copy link
Copy Markdown
Owner

Hi! Thank you for your contribution!

Copy link
Copy Markdown
Owner

@LeperGnome LeperGnome left a comment

Choose a reason for hiding this comment

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

lgtm

tea "github.com/charmbracelet/bubbletea"
"os"
"os/exec"
runtime "runtime"
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I think this could be just "runtime"

@LeperGnome LeperGnome merged commit dd37f8a into LeperGnome:main Sep 14, 2025
1 check passed
@LeperGnome
Copy link
Copy Markdown
Owner

Was not expecting to see you making MR to my project. Pleasant surprise! @jarpex

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