Jens Widell
Jens Widell
Critic does have "moved code detection," activated by pressing 'm' while displaying a diff that includes the move. (This short-cut and others are "documented" at the bottom of the page.)
Git doesn't know what's moved where any more than Critic does; Git doesn't store that information. Both Git and Critic has logic for figuring this out from a given diff....
The `fields_from_user` configuration option should apparently be a function that takes a single argument, a `dbutils.User` object, and returns a dictionary matching the `fields` configuration option, typically with all fields...
Yeah, that's sort of not supported. :-/ What you'll have to do is delete stuff from the database manually, using the "psql" command-line utility (or some other way of accessing...
Critic actually assigns changes per path and commit, so assigning all the changes in (only) a certain commit to a user is entirely possible. Whether you want to do that,...
The main email address is not automatically included in the "git emails." I intend for it to be, but have never gotten around to implementing it. In http://critic-review.org/889fd938?review=76 I added...
That's what I did in the system at Opera too, when I first created the usergitemails table. :-) Created http://critic-review.org/r/105 for some improvements in this area.
I can't reproduce this behavior at https://critic-review.org/. That system redirects back to the page on which you clicked "Sign in" after you've successfully signed in. And this is certainly how...
Created https://critic-review.org/r/301 for reviewing and testing these changes.
I seldom click the file names when reviewing; I just press space to display the next file, scroll down one page, and mark the previous file as reviewed, all in...