[PR #29] [MERGED] Reduce crate size using `cargo diet` by soon to be 26MB #51

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

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

Base: main ← Head: main

📝 Commits (2)

8d3a437 Reduce crate size using cargo diet by soon to be 26MB
354da53 Actually remove the 'hn-tui.toml' include which is overruled later

📊 Changes

3 files changed (+2 additions, -1 deletions)

View changed files

📝 Cargo.toml (+1 -0)
📝 src/hn-tui-default.toml (+0 -0)
📝 src/main.rs (+1 -1)

📄 Description

Here is the output it produced.

hackernews-TUI git:(main) cargo diet
┌───────────────────────────────────────┬─────────────┐
│ Removed File                          │ Size (Byte) │
├───────────────────────────────────────┼─────────────┤
│ run_debug                             │          93 │
│ .gitignore                            │         177 │
│ examples/hn-tui.toml                  │        2056 │
│ examples/assets/story_search_view.png │      153376 │
│ examples/assets/comment_view.png      │      160630 │
│ examples/assets/story_view.png        │      216828 │
│ examples/assets/help_view.png         │      293248 │
│ examples/assets/v0-5-demo-2.gif       │     9951136 │
│ examples/assets/v0-5-demo-1.gif       │    15331851 │
└───────────────────────────────────────┴─────────────┘
Saved 100% or 26.1 MB in 9 files (of 26.2 MB and 28 files in entire crate)

Please note that I found it easiest to move the default configuration
file that is included in main.rs into the root, similar to how
alacritty does it. There it might be more discoverable, too.

Also, please let me know if there is anything missing to make this
mergeable.

PS: Congrats to making it to the front page!

_{🔄 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/29 **Author:** [@Byron](https://github.com/Byron) **Created:** 4/25/2021 **Status:** ✅ Merged **Merged:** 4/25/2021 **Merged by:** [@aome510](https://github.com/aome510) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (2) - [`8d3a437`](https://github.com/aome510/hackernews-TUI/commit/8d3a4375a63dfcb01a112d65bad43f87b4b44195) Reduce crate size using `cargo diet` by soon to be 26MB - [`354da53`](https://github.com/aome510/hackernews-TUI/commit/354da53ed0ed36da24a7a847a8af05b27030d2cd) Actually remove the 'hn-tui.toml' include which is overruled later ### 📊 Changes **3 files changed** (+2 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.toml` (+1 -0) 📝 `src/hn-tui-default.toml` (+0 -0) 📝 `src/main.rs` (+1 -1) </details> ### 📄 Description Here is the output it produced. ``` hackernews-TUI git:(main) cargo diet ┌───────────────────────────────────────┬─────────────┐ │ Removed File │ Size (Byte) │ ├───────────────────────────────────────┼─────────────┤ │ run_debug │ 93 │ │ .gitignore │ 177 │ │ examples/hn-tui.toml │ 2056 │ │ examples/assets/story_search_view.png │ 153376 │ │ examples/assets/comment_view.png │ 160630 │ │ examples/assets/story_view.png │ 216828 │ │ examples/assets/help_view.png │ 293248 │ │ examples/assets/v0-5-demo-2.gif │ 9951136 │ │ examples/assets/v0-5-demo-1.gif │ 15331851 │ └───────────────────────────────────────┴─────────────┘ Saved 100% or 26.1 MB in 9 files (of 26.2 MB and 28 files in entire crate) ``` Please note that I found it easiest to move the default configuration file that is included in main.rs into the root, similar to how alacritty does it. There it might be more discoverable, too. Also, please let me know if there is anything missing to make this mergeable. PS: Congrats to making it to the front page! --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>