-
Notifications
You must be signed in to change notification settings - Fork 36
Blog post: Multipath Will Fix This #367
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
21dfef8
to
2c60a75
Compare
Direct link to the readable preview: https://iroh-computer-git-flub-multipath-will-fix-this-number-0.vercel.app/blog/multipath-will-fix-this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love the article.
Left some comments.
Otherwise looks good to me :)
I can highly vouch for just literally pasting the .mdx file into claude.ai and asking it to proof-read it and fix grammar and typos with a minimal diff, and then diff the produced file against the current one to fix any remaining typos. Works really well.
export const post = { | ||
draft: false, | ||
author: 'Floris Bruynooghe', | ||
date: '2025-07-29', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remember to fix the date, should it not get published today.
draft: false, | ||
author: 'Floris Bruynooghe', | ||
date: '2025-07-29', | ||
title: 'Multipath Will Fix This', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this title is basically an in-joke of the team. Which isn't even explained in the post. While that's a great working title, and is probably fine as a URL slug, I'm not sure this works as an actual title.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about "Iroh loves QUIC Multipath" (or even more cheeky "Iroh would love QUIC Multipath")?
Or something more dull like "Why we Plan to Implement QUIC Multipath".
Or "Why we're upgrading iroh's networking engine: Our plans with QUIC Multipath" (this could also be the description instead as it's really long).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used part of your suggestion as description 😄 I'm fine with what there is now, but probably won't object if someone feels like they have a better title/description combo.
Ok, this is basically ready from my side. Please proof-read once more. Fix inconsistencies and grammar, spelling. But maybe let's leave the general structure and contents in place. |
Pushed a commit that fixes all typos @flub |
Makes more sense after the first draft.
This looks much better.
b96d526
to
ac90f87
Compare
This doesn't really explain the title, probaly should be a different title. I'm also not really sure who the audience is. It's written from a very high level, hinting at the mechanisms involved without going into technical details.
It's also a first draft. But still, please comment.