🦀 THE GREAT OXIDATION: Converting this Go-pher into a Fearless Crab 🦀#2
Open
viemmsakh wants to merge 5 commits into
Open
🦀 THE GREAT OXIDATION: Converting this Go-pher into a Fearless Crab 🦀#2viemmsakh wants to merge 5 commits into
viemmsakh wants to merge 5 commits into
Conversation
Rust conversion
Owner
|
Tsk tsk... merge conflicts 🙄😏. |
Resolve Merge Conflits
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
Let’s be honest: Go is fine if you enjoy checking
if err != nilevery three lines, but we’re living in the future now. I have taken the liberty of rewritingkillahtaskin Rust, the only language that actually loves you back.This rewrite (internally dubbed RKT: Rusty Killah Task) provides the same CLI CRUD operations you know and love, but with 100% more borrow-checking and 0% garbage collection pause.
EDIT
Resubmitting PR with current merge conflicts identified in #1 resolved.
Why this is objectively better:
killahtask_<user>.csvnaming convention because I’m a benevolent Rustacean who respects the data of the past.Key Changes (The "Rusty" Way)
main.rs: Swapped out Cobra for Clap. Why? Because clapping is what people do when they see code this beautiful.file_handler/mod.rs: We still use CSVs, but now we usefs2for exclusive locking because we don't just "flock", we lock with intent.commands/mod.rs: Replaced the standard Go list logic withprettytable. Now your To-Do list actually looks like it belongs in the 21st century.helper/mod.rs: Using thewhoamicrate to find your username, because Rust knows exactly who you are (and that you deserve better than Go).Quips for the Reviewer
How to build (The Superior Way)
# Don't even bother with 'go build'. Experience the glory: cargo build --release ./target/release/rkt list