ReviewNB / support

Issues and feature requests for ReviewNB
https://reviewnb.com
59 stars 8 forks source link

can you support commenting when commit like when pull request?? #93

Closed planetb113 closed 3 years ago

amit1rrr commented 3 years ago

Sorry we're unable to understand the issue. Can you please describe the problem a little more?

planetb113 commented 3 years ago

Sorry we're unable to understand the issue. Can you please describe the problem a little more? 截屏2021-06-20 下午2 25 44 截屏2021-06-20 下午2 23 59

I mean reviewnb support rich diffs and commenting when review a pull request, but only support rich diffs in commit, can you support commenting when commit like when pull request??

@amit1rrr

amit1rrr commented 3 years ago

Thanks for the explanation.

By design, we only allow 2 type of comments -

  1. Pull Request Comments - Typical software development workflow wherein a set of changes are reviewed until they're approved for merge.
  2. JDoc comments - GoogleDoc style comments on notebook itself. These are good for a quick comment or discussion with non-technical stakeholders who typically don't care about the diff view.

We believe these 2 types of notebook comments cover all the collaboration requirements of data science teams. Please let me know if there's any workflow that doesn't fit either of these.

Now here's the main issue with Commit level comments -