5 ms·

My team has just started looking for a github replacement because the code review workflow is just not working for us, we need something with more structure. I

by guelo 11y ago

My team has just started looking for a github replacement because the code review workflow is just not working for us, we need something with more structure. I think there's plenty of space for feature and price competition, especially for private repos where github's social network effects don't matter as much.

farkas 11y ago
I'm the CEO of Atlassian (who run Bitbucket & Stash). Would love to get your feedback on what we can do to improve. We really pride ourselves on making products that have the right level of control in our products. Or email me - scott <at> atlassian.
fezzik02 11y ago
Hi! I'm a Developer Advocate over at Atlassian, and I'd like to point out that Stash offers extensible merge controls through merge request check plugins. In addition to pre-configured and third-party plugins, an open API allows you to make custom merge requirements. https://developer.atlassian.com/stash/docs/latest/tutorials-and-examples/controlling-when-pull-requests-can-be-merged.html https://developer.atlassian.com/stash/docs/latest/tutorials-... In Bitbucket, we don't support this yet, but we do allow branch restrictions to ensure only trusted users are able to merge sensitive branches and many teams find this to be a suitable proxy for that behavior. https://blog.bitbucket.org/2013/09/16/take-control-with-branch-restrictions/ https://blog.bitbucket.org/2013/09/16/take-control-with-bran...
Matt138 11y ago
Have you checked out any of the GitHub plugins that do this, something like https://www.zenhub.io https://www.zenhub.io could help you add more structure to GitHub
skeeterbug 11y ago
Doesn't BitBucket/Stash provide more structure for code reviews? You can lock down the main branch and force code to get merged via a PR.
meric 11y ago
We use gerrit hosted on EC2 mirroring commits to github for backup.
- RyJones 11y ago
  I am an unabashed fan of Gerrit - the fine-grained ACLs can be annoying to wrap your head around, but the code review interface is quite nice. the only thing I've found missing is the ability to render images inline. If someone submits a documentation update with SVG and PNG updates, you have to download the changeset to see the files. It would be nice if an option for "view this attachment inline" were available without installing extensions.
kibwen 11y ago
Github's code review capabilities are a huge weak point for team projects, and I've seen several services spring up to attempt to capitalize on this. Mozilla's Servo project is currently trialing one called Reviewable, here's a demonstration: https://reviewable.io/reviews/servo/servo/6456 https://reviewable.io/reviews/servo/servo/6456
halayli 11y ago
check out stash by atlassian https://www.atlassian.com/software/stash https://www.atlassian.com/software/stash
- kodablah 11y ago
  In my experience Stash is not much better. It has simple commenting on diffs, but we had to have a plugin to automatically "unapprove" everyone once a new commit was added to a PR. It's social aspects are no better than GitHub code comments either. I always found Crucible much nicer and it integrates w/ Stash decently, but I worry it and Fisheye are on the backburner at Atlassian.
  - farkas 11y ago
    What is it that you're after? You already know about Stash's plugin system that you can configure to do anything weird and wonderful (huge perk), but wondering if there is something we should bake into the core product? (Atlassian CEO here)
    - kodablah 11y ago
      So glad you responded, thank you. I definitely have suggestions. Our devops team was going to remove Crucible/Fisheye and I gave them this non-exhaustive list of why I would like to keep those tools in conjunction with stash: * Fisheye has crazy advanced searching (in commit messages, users, etc) * Fisheye offers he ability to watch for changes on files and directories that happen in any branch * Fisheye shows a file as it might look with all branches and tags together (though it doesn't always combine them well of course as might be expected w/out using the VCS's conflict resolution) * Crucible shows you what changed since you last reviewed and has an awesome slider (though can be a bit buggy) * Crucible live-notifies updates in the browser on changes * Crucible offers reviews on patches to a repo that may not be committed * Crucible differentiates comments and defects * Crucible lets me see unread vs read comments on files * Crucible lets me see the number of comments (read and unread) on in the file-list pane on the left before going to the file I personally think all 3 tools serve different purposes. I understand the want to move away from Crucible/Fisheye which are older and predate Git which causes some issues. IMO, there is a big unsatisfied market for code review tooling right now. There is also a market (probably not as big) for intelligent searching and notifications across a repo. Edit: Formatting
marcinkuzminski 11y ago
Maybe you should look at RhodeCode, it has several different workflows for code-review. From very simple like just voting on each individual commit, to a fully blown server side mergable workflow that includes, voting, checks by CI, and integration to external services. We did a blog about it recently on how we use it internally: https://rhodecode.com/blog/increased-automation-at-rhodecode/ https://rhodecode.com/blog/increased-automation-at-rhodecode... Displaimer: i'm the CTO of RhodeCode
- Zhenya 11y ago
  Displaimer = Displayed Disclaimer?
_yy 11y ago
Take a look at Phabricator: http://phabricator.org/ http://phabricator.org/ It has it's own code review workflow which is, in my opinion, superior to Github's. A nice summary: http://cramer.io/2014/05/03/on-pull-requests/ http://cramer.io/2014/05/03/on-pull-requests/
lmm 11y ago
What kind of structure? Bitbucket/Stash have explicit "approve" built in, which has always been plenty for my use case.
- jobvandervoort 11y ago
  FYI We've added 'Approve' to GitLab Enterprise Edition as well with the latest release (7.12).
haynes 11y ago
Hi guelo, can you elaborate a bit more what you are looking for? What kind of structure do you need? Without these informations, I'll just throw gitlab in the possible alternatives pool. https://about.gitlab.com/features/ https://about.gitlab.com/features/ Please note that I'm biased since I'm on the gitlab core team. :)

by guelo 11y ago

farkas 11y ago

I'm the CEO of Atlassian (who run Bitbucket & Stash). Would love to get your feedback on what we can do to improve. We really pride ourselves on making products that have the right level of control in our products. Or email me - scott <at> atlassian.

fezzik02 11y ago

Hi! I'm a Developer Advocate over at Atlassian, and I'd like to point out that Stash offers extensible merge controls through merge request check plugins. In addition to pre-configured and third-party plugins, an open API allows you to make custom merge requirements. https://developer.atlassian.com/stash/docs/latest/tutorials-and-examples/controlling-when-pull-requests-can-be-merged.html https://developer.atlassian.com/stash/docs/latest/tutorials-... In Bitbucket, we don't support this yet, but we do allow branch restrictions to ensure only trusted users are able to merge sensitive branches and many teams find this to be a suitable proxy for that behavior. https://blog.bitbucket.org/2013/09/16/take-control-with-branch-restrictions/ https://blog.bitbucket.org/2013/09/16/take-control-with-bran...

Matt138 11y ago

Have you checked out any of the GitHub plugins that do this, something like https://www.zenhub.io https://www.zenhub.io could help you add more structure to GitHub

skeeterbug 11y ago

Doesn't BitBucket/Stash provide more structure for code reviews? You can lock down the main branch and force code to get merged via a PR.

meric 11y ago

We use gerrit hosted on EC2 mirroring commits to github for backup.

RyJones 11y ago
I am an unabashed fan of Gerrit - the fine-grained ACLs can be annoying to wrap your head around, but the code review interface is quite nice. the only thing I've found missing is the ability to render images inline. If someone submits a documentation update with SVG and PNG updates, you have to download the changeset to see the files. It would be nice if an option for "view this attachment inline" were available without installing extensions.

kibwen 11y ago

Github's code review capabilities are a huge weak point for team projects, and I've seen several services spring up to attempt to capitalize on this. Mozilla's Servo project is currently trialing one called Reviewable, here's a demonstration: https://reviewable.io/reviews/servo/servo/6456 https://reviewable.io/reviews/servo/servo/6456

halayli 11y ago

check out stash by atlassian https://www.atlassian.com/software/stash https://www.atlassian.com/software/stash

kodablah 11y ago
In my experience Stash is not much better. It has simple commenting on diffs, but we had to have a plugin to automatically "unapprove" everyone once a new commit was added to a PR. It's social aspects are no better than GitHub code comments either. I always found Crucible much nicer and it integrates w/ Stash decently, but I worry it and Fisheye are on the backburner at Atlassian.
- farkas 11y ago
  What is it that you're after? You already know about Stash's plugin system that you can configure to do anything weird and wonderful (huge perk), but wondering if there is something we should bake into the core product? (Atlassian CEO here)
  - kodablah 11y ago
    So glad you responded, thank you. I definitely have suggestions. Our devops team was going to remove Crucible/Fisheye and I gave them this non-exhaustive list of why I would like to keep those tools in conjunction with stash: * Fisheye has crazy advanced searching (in commit messages, users, etc) * Fisheye offers he ability to watch for changes on files and directories that happen in any branch * Fisheye shows a file as it might look with all branches and tags together (though it doesn't always combine them well of course as might be expected w/out using the VCS's conflict resolution) * Crucible shows you what changed since you last reviewed and has an awesome slider (though can be a bit buggy) * Crucible live-notifies updates in the browser on changes * Crucible offers reviews on patches to a repo that may not be committed * Crucible differentiates comments and defects * Crucible lets me see unread vs read comments on files * Crucible lets me see the number of comments (read and unread) on in the file-list pane on the left before going to the file I personally think all 3 tools serve different purposes. I understand the want to move away from Crucible/Fisheye which are older and predate Git which causes some issues. IMO, there is a big unsatisfied market for code review tooling right now. There is also a market (probably not as big) for intelligent searching and notifications across a repo. Edit: Formatting

marcinkuzminski 11y ago

Maybe you should look at RhodeCode, it has several different workflows for code-review. From very simple like just voting on each individual commit, to a fully blown server side mergable workflow that includes, voting, checks by CI, and integration to external services. We did a blog about it recently on how we use it internally: https://rhodecode.com/blog/increased-automation-at-rhodecode/ https://rhodecode.com/blog/increased-automation-at-rhodecode... Displaimer: i'm the CTO of RhodeCode

Zhenya 11y ago
Displaimer = Displayed Disclaimer?

_yy 11y ago

Take a look at Phabricator: http://phabricator.org/ http://phabricator.org/ It has it's own code review workflow which is, in my opinion, superior to Github's. A nice summary: http://cramer.io/2014/05/03/on-pull-requests/ http://cramer.io/2014/05/03/on-pull-requests/

lmm 11y ago

What kind of structure? Bitbucket/Stash have explicit "approve" built in, which has always been plenty for my use case.

jobvandervoort 11y ago
FYI We've added 'Approve' to GitLab Enterprise Edition as well with the latest release (7.12).

haynes 11y ago

Hi guelo, can you elaborate a bit more what you are looking for? What kind of structure do you need? Without these informations, I'll just throw gitlab in the possible alternatives pool. https://about.gitlab.com/features/ https://about.gitlab.com/features/ Please note that I'm biased since I'm on the gitlab core team. :)