[PR #21] [MERGED] [#20] add config support #46

New issue

Closed

opened 2026-03-14 12:29:19 +03:00 by kerem · 0 comments

kerem commented

2026-03-14 12:29:19 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/hackernews-TUI/pull/21
Author: @aome510
Created: 4/13/2021
Status: ✅ Merged
Merged: 4/13/2021
Merged by: @aome510

Base: main ← Head: 20-add-config-support

📝 Commits (10+)

45c5434 add once_cell, toml
769d75a set up config file parsing from command line argument
9830b03 clean up imports
6c8977b add dirs
e91cc51 add default config file path
b0286eb use error! for errors
bb2bef0 use info for info
7d61764 allow to config story_polling
4381aee remove theme config
ed4147c use custom color only

📊 Changes

16 files changed (+391 additions, -92 deletions)

View changed files

📝 Cargo.lock (+148 -5)
📝 Cargo.toml (+17 -13)
➕ src/config.rs (+70 -0)
📝 src/hn_client.rs (+9 -6)
📝 src/main.rs (+70 -4)
📝 src/prelude.rs (+10 -11)
📝 src/view/async_view.rs (+2 -0)
📝 src/view/comment_view.rs (+10 -9)
📝 src/view/error_view.rs (+1 -0)
📝 src/view/help_view.rs (+3 -3)
📝 src/view/list_view.rs (+5 -5)
📝 src/view/search_view.rs (+8 -6)
📝 src/view/story_view.rs (+22 -16)
📝 src/view/theme.rs (+10 -5)
📝 src/view/utils.rs (+6 -2)
➖ theme.toml (+0 -7)

📄 Description

Resolve #20.

Brief summary of changes

add basic command line arguments
add support for parsing config file
code cleanup (imports cleanup, naming changes, etc)

_{🔄 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/21 **Author:** [@aome510](https://github.com/aome510) **Created:** 4/13/2021 **Status:** ✅ Merged **Merged:** 4/13/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `main` ← **Head:** `20-add-config-support` --- ### 📝 Commits (10+) - [`45c5434`](https://github.com/aome510/hackernews-TUI/commit/45c5434151993f35445e832f1316b8198b99edd3) add once_cell, toml - [`769d75a`](https://github.com/aome510/hackernews-TUI/commit/769d75a8a1713d8027a43d39a33377cfa3050436) set up config file parsing from command line argument - [`9830b03`](https://github.com/aome510/hackernews-TUI/commit/9830b036d5ba7767adc5594abccd19afd43ea9ab) clean up imports - [`6c8977b`](https://github.com/aome510/hackernews-TUI/commit/6c8977b17968fc083a2aef010752301035ba2f55) add dirs - [`e91cc51`](https://github.com/aome510/hackernews-TUI/commit/e91cc51c6055518515090170bdfe4d54486c8c98) add default config file path - [`b0286eb`](https://github.com/aome510/hackernews-TUI/commit/b0286eb568a897f3829759d5a9f45aa3e26a3624) use error! for errors - [`bb2bef0`](https://github.com/aome510/hackernews-TUI/commit/bb2bef058fd8d1cd9de919448d2e70b481b9b9ca) use info for info - [`7d61764`](https://github.com/aome510/hackernews-TUI/commit/7d6176440ff47d40e83e2053c2f4aa371a110921) allow to config story_polling - [`4381aee`](https://github.com/aome510/hackernews-TUI/commit/4381aee503bc58f0027ba2eec8190c20e34b07da) remove theme config - [`ed4147c`](https://github.com/aome510/hackernews-TUI/commit/ed4147c94add153be7677dbcf01e1cee7e02c4eb) use custom color only ### 📊 Changes **16 files changed** (+391 additions, -92 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+148 -5) 📝 `Cargo.toml` (+17 -13) ➕ `src/config.rs` (+70 -0) 📝 `src/hn_client.rs` (+9 -6) 📝 `src/main.rs` (+70 -4) 📝 `src/prelude.rs` (+10 -11) 📝 `src/view/async_view.rs` (+2 -0) 📝 `src/view/comment_view.rs` (+10 -9) 📝 `src/view/error_view.rs` (+1 -0) 📝 `src/view/help_view.rs` (+3 -3) 📝 `src/view/list_view.rs` (+5 -5) 📝 `src/view/search_view.rs` (+8 -6) 📝 `src/view/story_view.rs` (+22 -16) 📝 `src/view/theme.rs` (+10 -5) 📝 `src/view/utils.rs` (+6 -2) ➖ `theme.toml` (+0 -7) </details> ### 📄 Description Resolve #20. **Brief summary of changes** - add basic command line arguments - add support for parsing config file - code cleanup (imports cleanup, naming changes, etc) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>