Garen
-
2 votes
Garen
shared this idea and gave it 2 votes
·
-
3 votesstarted ·
AdminRoy
(Admin, Code Collaborator)
responded
This is already in the design for 7
Garen
shared this idea and gave it 3 votes
·
-
17 votes
Garen
shared this idea and gave it 1 vote
·
-
23 votes
Garen
gave this 1 vote
·
Garen
commented
·
I would like to see a visual indication at the top of the review as to what the current phase is. Likewise, the button at the bottom of the phase is also not clear sometimes. It just says "Finish". Finish what? The most frequent question I get asked is "how do we REALLY finish a review?" Also, I've had so many reviews open that I clicked "Finish" on the wrong review before!
-
10 votes
Garen
commented
·
Would want to see that a particular branch/version of a file was code-reviewed before it was merged to the mainline ("trunk", /main branch, integration branch etc). It could then be an automated aid to ensure all code that made it into mainline or a release had been code reviewed.
Garen
shared this idea and gave it 3 votes
·
-
164 votescompleted ·
AdminRoy
(Admin, Code Collaborator)
responded
Garen
commented
·
My point about branches was to highlight the need to cherry-pick specific versions via the GUI, which it cannot do--the GUI will only upload the version presently selected and it's predecessor. Users need to be able to use a GUI to pick the exact versions they want to upload. In my previous example, the use case was for the tip of a branch verses the version it descended from.
Garen
commented
·
Note: an "activity" is UCM specific. ClearCase supports two modes of operation: base and UCM. We don't use UCM, and so don't care about UCM support at all.
Garen
commented
·
We have to tell everyone to use the command line client and use "addversions". Most people work on an isolated branch, so they don't care about uploading foo.c@@/main/bar/2 and the predecessor foo.c@@/main/bar/1 -- which is the default tray/GUI behavior. If anything they want to upload the latest changes from branch /bar, and the latest version on the /main branch.
Garen
gave this 3 votes
·
Argh, I meant to say "the button at the bottom of the page is also not clear sometimes."