jeffkaufman / nomic

Automated nomic
18 stars 11 forks source link

Dashboard shows need for re-review when it's not necessary #227

Closed tnelling closed 5 years ago

tnelling commented 5 years ago

Result of merging #222

Example: I've approved #225, and the build output reflects this, but the dashboard says I need to re-review it.

jeffkaufman commented 5 years ago

This is going to be annoying to fix.

I need to get "has the diff changed" available to the JS.

jeffkaufman commented 5 years ago

I think this is fixed now; let me know if you run into any problems.