Closed bzz closed 5 years ago
~,~ means removing a comma
😮 Thank you so much for a thoughtful and thorough review! Will address asap 🚀
Thank you again, @vmarkovtsev feedback addressed in e9127ee
@campoy
Hey, could you break the long lines into shorter easier to review chunks?
Done, although I understand the rationale, for the plain text this requirement seems quite un-friendly to different screen/font sizes, modern editors, semantics of the text and tools like Grammarly, no? 😕
Ready for another pass.
Sorry about the inconvenience, @bzz. I agree it's a pain to limit the line length but otherwise reviews become basically impossible.
Any other improvements to the current flow are welcome.
Looking good, @bzz!
Thank you very much @campoy, appreciate your feedback! All suggestions make sense to me, will start applying.
As soon as it's done will ping you and push it to staging for preview.
All feedback has been addressed, deployed on staging to https://blog-staging.srcd.run/post/review-building-static-analysis-tools/
@campoy ready for another pass
@vcoisne @eiso Since Francesc went on vacations, maybe you could review this instead please?
all previous comments were addressed
@campoy friendly ping
@campoy @vcoisne appreciate you feedback, it's been addressed, re-deployed on staging, ready for another round.
As it's been almost 2 months we are working on this, it would be nice to try to wrap it up soon.
LGTM I think it's ready to be published early next week.
@bzz scheduled for next Thursday as you can see in the "Blog & Press" google calendar I created
Excited to see this published today.
Me to! :shipit:
When it's done will also submit a medium post to the source{d} publication.
First "paper review" blog post. Cross-post from medium
Closes #213