How can we improve Azure DevOps?

TFS code review should not require a new code review to be started when code that is already under review is changed based on review comment

This suggestion is migrated to Developer Community. Please use below link to view the current status.
It seems that TFS requires a new code review to be started whenever code that is already under review is updated in response to review comments. It is not only unnecessary busy work for a developer to initiate a new review each time a review comment would suggest a change to the code that is already under review is necessary, but it also prevents having a single picture of changes made as a result of the review process.

With the current method, a single logical code review may span multiple physical code reviews and the comments for the single logical review are fragmented across several physical reviews.

1,137 votes
Sign in
Password icon
Signed in as (Sign out)

We’ll send you updates on this idea

Michael shared this idea  ·   ·  Flag idea as inappropriate…  ·  Admin →


Sign in
Password icon
Signed in as (Sign out)
  • Sergio Ramos commented  ·   ·  Flag as inappropriate

    I'm forced to use TFVC because it's exclusive for our ERP, and i had no choice to buy a third party for code review...

  • Anonymous commented  ·   ·  Flag as inappropriate

    Good question Rahul Shinde. What is your roadmap for TFVC? Should we be moving our code to GIT?

  • Rahul Shinde commented  ·   ·  Flag as inappropriate

    VSTS team, it looks like Git is more of priority than TFVC. Are you recommending to the community to not use TFVC in future?

  • Alex commented  ·   ·  Flag as inappropriate

    another 6 months have passed and still no updates on when this will happen

  • Alex commented  ·   ·  Flag as inappropriate

    The Pull request feature for Git looks a lot more promising tool for effective code reviews, it would be good to hear if plans still exist to enhance the TFVC code review experience in similar ways.

  • Anonymous commented  ·   ·  Flag as inappropriate

    Is there any update on implementation target for this request for TFVC? Thanks in advance.

  • pipja commented  ·   ·  Flag as inappropriate

    I love Git's code review, if I made some changes to the code review, the reviewer only need to view the new commits to see what changed since. And then can view the whole change in its entirety if need be. If you are going to kill TFSVC, please do announce it now so we can move on.

  • Sina Jazayeri commented  ·   ·  Flag as inappropriate

    The problem with reviewing shelvesets is lack of traceability as shelvesets can be deleted at any time. This is problem if you have to show review of code for audit/process requirements.

  • Ale commented  ·   ·  Flag as inappropriate

    Taken from a different source:

    Here are the steps necessary to update the shelveset associated with the review.

    1. From the "Code Review" pane select the "view shelveset" link
    2. From the "Shelveset Details" pane highlight and copy the shelveset name
    3. Navigate to the "Pending Changes" pane, click on "Shelve" and paste the shelveset name
    4. Press the Yes button on the shelveset replace verification dialog
    5. Now the reviewer can see the updated files and the review discussion can continue

  • Andrew Stanton commented  ·   ·  Flag as inappropriate

    @Visual Studio Tea - Git is stupidly dangerous. I don't want to have the ability to "whoops" an entire source control history. The point of having version control is so I don't lose changes.

    The only time I need to use the "powerful" features of Git is to unfuck myself when I was trying to do things that should be brainless and missed some convoluted `--i -o --git-******` switch or vim command resulting in an "oh ****" moment.

    Programming well is hard enough. Stop pandering to that ****** source control before your own, unless you are going to fix it's deficiencies.

  • Sina Jazayeri commented  ·   ·  Flag as inappropriate

    For those waiting, we gave up waiting and invested in Smatbear Collaborater which offers iterative changes and integrates well with TFS. It's clear TFVC is no longer treated as high priority by MS.

  • Noel commented  ·   ·  Flag as inappropriate

    This is so pathetic, you need to implement this immediately. We have been waiting forever.

  • Doug Waterfield commented  ·   ·  Flag as inappropriate

    I have recently begun leading a team that has been using TFSVC for a while. I'm really surprised that such a basic workflow isn't available!

    When will this be in the product?

  • Michael Denholm commented  ·   ·  Flag as inappropriate

    Considering its now 15 months later and a new version of TFS recently released, should I assume this will never be implemented in TFVC, give up on it and move to Git?

  • Anonymous commented  ·   ·  Flag as inappropriate

    You are too late! Who in their right mind will use a CR tool that requires a new CR to be started after reflecting even a single comment ???

  • Rob Siklos commented  ·   ·  Flag as inappropriate

    @Andrew Ch: This is not a a reliable of doing things. The problem is that as the comments get addressed, the line numbers change, and you complete lose sight of what the original code review captured.

    @80's Rocker: Couldn't agree more, but the writing is clearly on the wall - TFSVC is dead, and too bad for all of us that are using it. I guess we're just expected to migrate to Git or stay with TFSVC and live with a reduced feature set. I wish they'd just come right out and say this. At the end of the day, it seems the mob has spoken and Git is the winner.

  • 80's Rocker commented  ·   ·  Flag as inappropriate

    This is really sad that it on 1/15/16 this was still not on the roadmap to be implemented. Greate feature is being killed by poor implementation and not fixing a glaring feature shortcoming in a timeley mannner. The VS team obviosly uses GIT and that is why getting this implemented in GIT was a big priority instead of TFS. But by doing that they are ignoring a very big user base of TFS who don'want to switch over to GIT.

← Previous 1 3 4

Feedback and Knowledge Base