Uploaded image for project: 'Bitbucket Cloud'
  1. Bitbucket Cloud
  2. BCLOUD-7968

Ability to approve files individually in a pull request (BB-9145)

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      It's hard to review large pull requests with a lot of files. So it would be nice to mark as approved some files that you are sure that are ok. It would be nicer if these files that you already reviewed, be collapsed so you didn't have to look at them anymore, unless they have changed again.

      A mark on the side of the files that I approve on the list of changed files would be nice too.

       


      Latest update from Atlassian 2023-10-06

      Bitbucket Cloud has shipped a feature allowing pull request files to be marked as viewed (and therefore visually collapsed): 

      https://community.atlassian.com/t5/Bitbucket-articles/Keep-track-of-viewed-files-on-Bitbucket-Cloud-pull-requests/ba-p/2497882

            [BCLOUD-7968] Ability to approve files individually in a pull request (BB-9145)

            Pinned comments

            Hi all, happy to say we've just released the ability to mark files as viewed on pull requests. Read all about it here!

            https://community.atlassian.com/t5/Bitbucket-articles/Keep-track-of-viewed-files-on-Bitbucket-Cloud-pull-requests/ba-p/2497882

            Dave Parrish [Atlassian] added a comment - Hi all, happy to say we've just released the ability to mark files as viewed on pull requests. Read all about it here! https://community.atlassian.com/t5/Bitbucket-articles/Keep-track-of-viewed-files-on-Bitbucket-Cloud-pull-requests/ba-p/2497882

            All comments

            Hi all, happy to say we've just released the ability to mark files as viewed on pull requests. Read all about it here!

            https://community.atlassian.com/t5/Bitbucket-articles/Keep-track-of-viewed-files-on-Bitbucket-Cloud-pull-requests/ba-p/2497882

            Dave Parrish [Atlassian] added a comment - Hi all, happy to say we've just released the ability to mark files as viewed on pull requests. Read all about it here! https://community.atlassian.com/t5/Bitbucket-articles/Keep-track-of-viewed-files-on-Bitbucket-Cloud-pull-requests/ba-p/2497882

            Is Atlassian developing features for Bitbucket Cloud AT ALL?

            Robert Evans added a comment - Is Atlassian developing features for Bitbucket Cloud AT ALL?

            +1

            Anders Hallin added a comment - +1

            Andrea added a comment -

            +1

            Andrea added a comment - +1

            Matteo added a comment -

            +1

            Matteo added a comment - +1

            +1

            +1

            Laureano Hess added a comment - +1

            +1

            +1

            +1

            +1

            Jernej Gosar added a comment - +1

            Jason Kobs added a comment -

            Agreed - this is the only thing preventing us from migrating off of Azure DevOps over to BitBucket.

            Jason Kobs added a comment - Agreed - this is the only thing preventing us from migrating off of Azure DevOps over to BitBucket.

            it has been 10 years sinces it was originallyc reated. I has 122 votes, how many are requiered for it to get developed?

            Pablo Szittyay added a comment - it has been 10 years sinces it was originallyc reated. I has 122 votes, how many are requiered for it to get developed?

            Jason Kobs added a comment -

            +1

            Jason Kobs added a comment - +1

            +1

            +1

            +1

            +1

            soportedecom added a comment - +1

            While you gather interest, people are moving in github or azure devops.

            Disappointing... 

            That's a deal breaker for us.

            Daniel Ivanov added a comment - While you gather interest, people are moving in github or azure devops. Disappointing...  That's a deal breaker for us.

            Marc Scherer added a comment - - edited

            Go and do it now Atlassian. It is critical for large teams and large pull requests.

            How else are we supposed to (thoroughly) review a pull request with 50+ files and 5+ reviewers?

            And no, it is not always possible to make smaller pull requests, even if that is desirable.

            Marc Scherer added a comment - - edited Go and do it now Atlassian. It is critical for large teams and large pull requests. How else are we supposed to (thoroughly) review a pull request with 50+ files and 5+ reviewers? And no, it is not always possible to make smaller pull requests, even if that is desirable.

            +1 Please consider this feature. GitHub has it since 2019
            https://github.blog/2019-07-01-mark-files-as-viewed/

            Pablo Szittyay added a comment - +1 Please consider this feature. GitHub has it since 2019 https://github.blog/2019-07-01-mark-files-as-viewed/

            + 1

            Mark Zieg added a comment -

            +1

            Mark Zieg added a comment - +1

            +1

            +1

            This ability in Azure DevOps is a major part of my workflow and it's hard to recommend to our DevOps team that we migrate to Bitbucket without it.

            Thomas Parikka added a comment - +1 This ability in Azure DevOps is a major part of my workflow and it's hard to recommend to our DevOps team that we migrate to Bitbucket without it.

            Thomas Morel added a comment - +1 See github implementation -> https://github.blog/2019-07-01-mark-files-as-viewed/

            +1

            +1

            +1

            +1

            Gustavo Binow added a comment - +1

            +1

            +1

            NickRattermann added a comment - +1

            yongzhy added a comment -

            +++1, 

            yongzhy added a comment - +++1, 

            Grant Pitel added a comment - - edited

            I figured I would would mention BSERV-4266 is very similar. 

            We were using SVN/Crucible (Atlassian product) which had this "marked at viewed" feature.  It was essential for keeping my position on reviews containing +50 file changes.

            Its sad that their flagship/supported product is missing the features. 

            Grant Pitel added a comment - - edited I figured I would would mention BSERV-4266 is very similar.  We were using SVN/Crucible (Atlassian product) which had this "marked at viewed" feature.  It was essential for keeping my position on reviews containing +50 file changes. Its sad that their flagship/supported product is missing the features. 

            +++1 Please make it happen

            Avi Mehenwal added a comment - +++1 Please make it happen

            +1

            +1

            Nick Monkman added a comment - +1

            +1

            Sam Winter added a comment -

            +++1

            Sam Winter added a comment - +++1

            +1   +1

            Ahmad Elqaoud added a comment - + 1   +1

            +1

            Rick Veenstra added a comment - +1

            +1

            Koushik M L N added a comment - +1

            +1 from me. When there are lots of files in a review, being able to mark individual files as "Done" would be great. I really don't mind how simple it is. Just some way to tick things off as I go.

            Kenny McCartney added a comment - +1 from me. When there are lots of files in a review, being able to mark individual files as "Done" would be great. I really don't mind how simple it is. Just some way to tick things off as I go.

            +1

             

            Mashaal Memon added a comment - - edited

            +1 - doesn't need to be complicated. Just mark file as "ok". if it isn't ok, would have a comment. Would help with large PRs submitted, and frankly even short ones!

            As the moment, all I can do is comment on files that are fine putting comments like "Looks good". My developers then have to go through each file, looking for actionable comments. Not a good workflow for them or myself. 

            Mashaal Memon added a comment - - edited +1 - doesn't need to be complicated. Just mark file as "ok". if it isn't ok, would have a comment. Would help with large PRs submitted, and frankly even short ones! As the moment, all I can do is comment on files that are fine putting comments like "Looks good". My developers then have to go through each file, looking for actionable comments. Not a good workflow for them or myself. 

            Jon added a comment -

            It's good to have a flag indicating per-file approved/not approved.  But I would be best to also have a (full! not just the most recent, in case you accidentally toggle the button) history of past approvals and on which commit the approval was given.  Otherwise, the flag becomes nearly useless if the user has to guess/remember what commit they'd approved up to. 

            For similar reasons, it will be critical to easily see which approved files in the PR have had subsequent commits without having to click around to find out.

            I really wish PRs themselves had this functionality (separate issue).  On long-running issues with many commits and review cycles it gets to be quite challenging to know where you left off last time.  But as long as each approve/unapprove is just another event recorded per file, that may be the most useful anyway.

            Jon added a comment - It's good to have a flag indicating per-file approved/not approved.  But I would be best to also have a (full! not just the most recent, in case you accidentally toggle the button) history of past approvals and on which commit the approval was given.  Otherwise, the flag becomes nearly useless if the user has to guess/remember what commit they'd approved up to.  For similar reasons, it will be critical to easily see which approved files in the PR have had subsequent commits without having to click around to find out. I really wish PRs themselves had this functionality (separate issue).  On long-running issues with many commits and review cycles it gets to be quite challenging to know where you left off last time.  But as long as each approve/unapprove is just another event recorded per file, that may be the most useful anyway.

            Tyler T added a comment -

            We are reopening this issue because this is a feature we plan to add in the near future. Stay tuned for updates and thank you for your patience!

            Tyler T added a comment - We are reopening this issue because this is a feature we plan to add in the near future. Stay tuned for updates and thank you for your patience!

            aforesti added a comment -

            Attachment 2013-08-22 10_54_27-e0ibkf5g.png has been added with description: Originally attached to Bitbucket issue #7968 in site/master

            aforesti added a comment - Attachment 2013-08-22 10_54_27-e0ibkf5g.png has been added with description: Originally attached to Bitbucket issue #7968 in site/master

            Issue BCLOUD-16222 was marked as a duplicate of this issue.

            Alastair Wilkes added a comment - Issue BCLOUD-16222 was marked as a duplicate of this issue.

            @acoates - That's definitely true in some cases. That said, commit-by-commit review wouldn't be mandatory; you'd be able to see changes across any range of commits in the PR (including the entire range, like what is shown today). This is especially. useful for "show me what's changed since I last reviewed." Either way, we don't plan to add file-level review states (where some files are approved and some aren't) any time soon.

            Alastair Wilkes added a comment - @acoates - That's definitely true in some cases. That said, commit-by-commit review wouldn't be mandatory; you'd be able to see changes across any range of commits in the PR (including the entire range, like what is shown today). This is especially. useful for "show me what's changed since I last reviewed." Either way, we don't plan to add file-level review states (where some files are approved and some aren't) any time soon.

            @awbb - Regarding commit-by-commit reviews - in an ideal world, everyone would craft beautiful commits that were logically isolated. But the reality is that commits in a branch often undo things that happened in earlier commits (ie. where the first implementation of an idea didn't quite work out), so reviewing on a commit basis would waste a lot of time.

            Deleted Account (Inactive) added a comment - @awbb - Regarding commit-by-commit reviews - in an ideal world, everyone would craft beautiful commits that were logically isolated. But the reality is that commits in a branch often undo things that happened in earlier commits (ie. where the first implementation of an idea didn't quite work out), so reviewing on a commit basis would waste a lot of time.

              Unassigned Unassigned
              01138428f423 aforesti
              Votes:
              152 Vote for this issue
              Watchers:
              87 Start watching this issue

                Created:
                Updated:
                Resolved: