-
Notifications
You must be signed in to change notification settings - Fork 37
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
add ipfs-check blog post #697
Conversation
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 32.8%, saving 23.33 KB.
593 images did not require optimisation. |
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.
thank you for amplifying this @2color! lgtm, cosmetics inline, merge when you want
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.
Made a few suggestions to simplify wording or turn longer paragraphs into lists, for easier scanning. None are blocking, merge when ready.
Co-authored-by: Mosh <[email protected]> Co-authored-by: Marcin Rataj <[email protected]>
I'm blocking publishing until we address ipfs/ipfs-check#70 |
We deployed a fix to the issue I mentioned. So this is good to go. |
No description provided.