[llvm-dev] Chrome extension for tracking LLVM code review

18 views
Skip to first unread message

River Riddle via llvm-dev

unread,
Mar 30, 2020, 4:44:47 PM3/30/20
to llvm-dev
Hi all,

Recently I've had some extra time on my hands, so I spent part of the weekend coding up a simple chrome extension to monitor the status of code reviews on the LLVM phabricator instance. This extension is inspired by some other review monitoring tools that I've used in the past. It allows for monitoring revisions that a user needs to review, revisions by the user that need to be updated, and revisions by the user that are accepted and ready to land. I'm posting here as this may be useful to others. I'm also happy to move this directly upstream if the community is interested.

(disclaimer: I have limited experience in javascript so some things could definitely be improved).

The code can be found here(along with an example screenshot of my own review status): https://github.com/River707/llvm-review-monitor 

Thanks
-- River

River Riddle via llvm-dev

unread,
Mar 31, 2020, 3:16:45 PM3/31/20
to llvm-dev
For those interested I've uploaded to the chrome store to make installs easier. An install link is now included on the repo.

-- River

Johannes Doerfert via llvm-dev

unread,
Mar 31, 2020, 11:45:27 PM3/31/20
to River Riddle, llvm-dev

I'd be interested in using this but I have to wait until someone ports this to firefox (or at least chromium).

Thanks for sharing it though!

_______________________________________________
LLVM Developers mailing list
llvm...@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

River Riddle via llvm-dev

unread,
Apr 1, 2020, 12:42:40 AM4/1/20
to Johannes Doerfert, llvm-dev
Hey Johannes,

It should work for chromium out-of-the-box(in fact I just verified on my windows machine). For firefox I uploaded a simple API fix and verified that it could run there as well. Chromium can use the same install link as chrome, for firefox I'll have to figure out how to upload an extension to the webstore.

-- River

Johannes Doerfert via llvm-dev

unread,
Apr 1, 2020, 1:33:58 AM4/1/20
to River Riddle, llvm-dev

Hi River,

Disclaimer: I'm really not well versed in the JavaScript web browser
wold, so I might just do things wrong.


I tried Chromium before, all I get is: "No revisions require your
attention."
Maybe I have to wait for something to change?

I tried to manually build it with: `npm run-script build` but that
results in some weird (apparently often occurring) socket issue during
webpack. Stack overflow is full of descriptions of the same issues (=
same error) but always different causes. After a few attempt to make the
build work I basically gave up. Interestingly, I somehow generated a
build folder with a manifest file that I could load into firefox.
However, the popup says "Unhandled error: TypeError: resp.result.data[0]
is undefined" and the inspector shows all kinds of errors. Again, this
is probably because of my half-broken build.


FWIW, you don't have to fix this for me :)

Cheers,
   Johannes

listllvm-dev@lists.llvm.orghttps://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

Reply all
Reply to author
Forward
0 new messages