Better code reviews for GitHub. It integrates with GitHub to provide a better reviewing and commenting experience for pull requests. Free for open source!
265 karma · joined January 3, 2019
Better code reviews for GitHub. It integrates with GitHub to provide a better reviewing and commenting experience for pull requests. Free for open source!
No newsletter yet, but you can follow me on Twitter for now @jameslao.
I've made a couple notes based on this to help me prioritize next. Thanks for taking the time to provide all this amazing feedback!
1. More comment states. Comments start as "open". The workflow I had in mind was that once the author addresses it, they would mark it as "resolved". The reviewer can go to the comments view (in the meta section on the left nav) where they can filter to resolved comments. If it looks good, they can mark it as "closed" or "open" again if they want more changes. I have this as a suggested workflow in the docs. [1]
2. The iterations section in the left nav lets you compare any two iterations. You can use this to see what has changed since your last review. Crocodile even indicates which iteration your last review was on to make this easy.
Another feature I've thought about is "show me all of person X's reviews and comments across all repos". That would have been super useful when I was on promo panels to go through and see if a promo candidate's review comments were useful, constructive, kind, etc.
Thanks for the feedback. I really appreciate it!
A couple improvements I made:
* You can see the whole discussion. The last time I used CodeFlow it only showed one line of the comment which often wasn't enough for me to remember what it was so in practice I always had to click on every single one to see the whole thing. Not a big deal with CodeFlow since it was so snappy as you mentioned.
* It shows a preview of the code that was commented on to provide context.
Thanks for taking a look! Great to see other CodeFlow fans out there.
Crocodile has its own organization and member list. Billing is based on the max number of users in the Crocodile org for the month.
Supporting other VCS providers, even non-Git ones, is on my list. The underlying data model is agnostic to GitHub so it's just a matter of having demand for it and doing the work!
Not right now. The suggestion feature is something I want to add in the future if there's interest though!
> Given your history with Microsoft and their internal tool which this is inspired by, how long until github copies some of the ideas?
No idea! But CodeFlow had been around for many years before they acquired GitHub so they've had plenty of time to do it! I think this is an existential fear that anyone building code review tools has. For Crocodile, I think the long term strategy will be to decouple this from GitHub so that it can work with any source control provider. I architected it this way from the beginning but decided to target GitHub first because they have mindshare and because it made it really easy to dogfood since I host my code on GitHub.
> Are there potential problems you see outside of current product?
There's the one you already mentioned, which is GitHub improving their PR system, but there's not much I can do about that. Current problems I'm looking at are roadmap related. I have a few ideas of features I'd like to add, but no real indication on what to do next. Hence, posting on HN to get feedback. :)
Thanks for taking a look and the feedback!
I also use Tailwind CSS and I found a way to make that work nicely with HTMX for CSS transitions which I wrote about [0].
[0] https://www.crocodile.dev/blog/css-transitions-with-tailwind...
- In door water dispensers and ice makers are unreliable and will likely break.
- Availability of repair technicians and spare parts varies by brand and is region dependent.
- Different brands are good at different appliances. For instance, Bosch is well known for their dishwashers. Unfortunately I didn't find one in my budget that clearly stood out from the rest.
Never hurts to talk to someone at a local appliance center if you have one nearby.