[PR #35] [MERGED] Add `article_parse_command` #55

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/aome510/hackernews-TUI/pull/35
Author: @aome510
Created: 5/29/2021
Status: ✅ Merged
Merged: 5/29/2021
Merged by: @aome510

Base: main ← Head: try-article-md

📝 Commits (7)

9ad8587 add ArticleParseCommand struct
3d0cdcb add article_parse_command field to default config file and its documentation
586283d allow to use custom defined article_parse_command
ac2814f add article_parse_command documentation to README and error message
ce4ffc9 use article_md instead of article-md-cli
98c2eb9 fix spelling
3da7d96 small tweak wordings

📊 Changes

5 files changed (+96 additions, -34 deletions)

View changed files

📝 README.md (+30 -16)
📝 src/config.rs (+11 -0)
📝 src/hn-tui-default.toml (+18 -0)
📝 src/view/article_view.rs (+30 -13)
📝 src/view/async_view.rs (+7 -5)

📄 Description

allow defining custom article parser command besides the default mercury-parser
handle invalid url when rendering ArticleView
modify documentation about the parser for ArticleView

_{🔄 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/35 **Author:** [@aome510](https://github.com/aome510) **Created:** 5/29/2021 **Status:** ✅ Merged **Merged:** 5/29/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `main` ← **Head:** `try-article-md` --- ### 📝 Commits (7) - [`9ad8587`](https://github.com/aome510/hackernews-TUI/commit/9ad858772874fd9ae8454daa32caae177f18dee0) add ArticleParseCommand struct - [`3d0cdcb`](https://github.com/aome510/hackernews-TUI/commit/3d0cdcb9514a125a92a269d20ac2a50625236b56) add article_parse_command field to default config file and its documentation - [`586283d`](https://github.com/aome510/hackernews-TUI/commit/586283d96f063bd92097d5cc434b9008775b3735) allow to use custom defined article_parse_command - [`ac2814f`](https://github.com/aome510/hackernews-TUI/commit/ac2814fba3e2e625a9f221db092d3af18bb9eae2) add article_parse_command documentation to README and error message - [`ce4ffc9`](https://github.com/aome510/hackernews-TUI/commit/ce4ffc9f4f5edac3d1c5b2c592a5411b82684141) use article_md instead of article-md-cli - [`98c2eb9`](https://github.com/aome510/hackernews-TUI/commit/98c2eb971b8be9bcc9597a3c509a38a37be486d8) fix spelling - [`3da7d96`](https://github.com/aome510/hackernews-TUI/commit/3da7d96748213b8972ecf0ae35946b02cac1461a) small tweak wordings ### 📊 Changes **5 files changed** (+96 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+30 -16) 📝 `src/config.rs` (+11 -0) 📝 `src/hn-tui-default.toml` (+18 -0) 📝 `src/view/article_view.rs` (+30 -13) 📝 `src/view/async_view.rs` (+7 -5) </details> ### 📄 Description - allow defining custom article parser command besides the default `mercury-parser` - handle invalid url when rendering `ArticleView` - modify documentation about the parser for `ArticleView` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>