[PR #88] [MERGED] Add cross-platform default value for url_open_command #95

Closed
opened 2026-03-14 12:32:01 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/hackernews-TUI/pull/88
Author: @aome510
Created: 4/19/2023
Status: Merged
Merged: 4/19/2023
Merged by: @aome510

Base: mainHead: cross-platform-open-command


📝 Commits (5)

  • 20a473e rename doc to docs
  • 46d6d96 make url_open_command have cross-platform default value
  • ffdfb8e update doc/ references to use docs/
  • 1f8ac17 fix typo
  • a1be3eb fix clippy errors

📊 Changes

8 files changed (+29 additions, -8 deletions)

View changed files

📝 README.md (+2 -2)
📝 docs/config.md (+9 -1)
📝 examples/hn-tui.toml (+3 -1)
📝 hackernews_tui/src/config/mod.rs (+11 -0)
📝 hackernews_tui/src/view/article_view.rs (+1 -1)
📝 hackernews_tui/src/view/async_view.rs (+1 -1)
📝 hackernews_tui/src/view/comment_view.rs (+1 -1)
📝 hackernews_tui/src/view/story_view.rs (+1 -1)

📄 Description

Resolves #87


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/aome510/hackernews-TUI/pull/88 **Author:** [@aome510](https://github.com/aome510) **Created:** 4/19/2023 **Status:** ✅ Merged **Merged:** 4/19/2023 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `main` ← **Head:** `cross-platform-open-command` --- ### 📝 Commits (5) - [`20a473e`](https://github.com/aome510/hackernews-TUI/commit/20a473e87c3f397f43ef061ba6472637fa478fb0) rename doc to docs - [`46d6d96`](https://github.com/aome510/hackernews-TUI/commit/46d6d969d668e228fcff2c251853a4bd8ee7aa1b) make `url_open_command` have cross-platform default value - [`ffdfb8e`](https://github.com/aome510/hackernews-TUI/commit/ffdfb8e2840d58413fabf29e3b02cd49dc593956) update `doc/` references to use `docs/` - [`1f8ac17`](https://github.com/aome510/hackernews-TUI/commit/1f8ac17bf7f1c1ca91ca212f7446b68969c8f465) fix typo - [`a1be3eb`](https://github.com/aome510/hackernews-TUI/commit/a1be3eb2becd5b2e54cc15ccffd3ad9a7716dc02) fix clippy errors ### 📊 Changes **8 files changed** (+29 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+2 -2) 📝 `docs/config.md` (+9 -1) 📝 `examples/hn-tui.toml` (+3 -1) 📝 `hackernews_tui/src/config/mod.rs` (+11 -0) 📝 `hackernews_tui/src/view/article_view.rs` (+1 -1) 📝 `hackernews_tui/src/view/async_view.rs` (+1 -1) 📝 `hackernews_tui/src/view/comment_view.rs` (+1 -1) 📝 `hackernews_tui/src/view/story_view.rs` (+1 -1) </details> ### 📄 Description Resolves #87 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-03-14 12:32:01 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hackernews-TUI#95
No description provided.