Presentation is loading. Please wait.

Presentation is loading. Please wait.

Miles Parker (Tasktop) Sebastien Dubois (Ericsson) EclipseCon 2013 Hey! What's wrong with Gerrit?!

Similar presentations


Presentation on theme: "Miles Parker (Tasktop) Sebastien Dubois (Ericsson) EclipseCon 2013 Hey! What's wrong with Gerrit?!"— Presentation transcript:

1 Miles Parker (Tasktop) Sebastien Dubois (Ericsson) EclipseCon 2013 Hey! What's wrong with Gerrit?!

2

3 Reviews are Eating the World Look Guys, More Code!!! Yummy! I love juicy bugs...

4 Limited Workspace Integration Reviews == Commits Minimal Structure No Visibility across Patch Sets, Reviews Poor Traceability Rigid Workflow Annoying Notifications The copyright header formatting isn't consistent... Um... you weren't planning to submit that, were you? What you really want here is a Publish-Subscribe pattern...

5 Review Navigator Review Artifacts and Comments outside of Editor! CNF based Filter and presentation controls

6 Editor Enhancements Section Re-Ordering Inline Global Comments Decorators (Lei Zhu) Spell Checking Wiki Text More Mylyn goodness!

7 Compare Editor Input Code Navigation, etc.. Ported from R4E

8 Why do I have to keep repeating myself?! Limited Workspace Integration Reviews == Commits Minimal Structure No Visibility across Patch Sets, Reviews Poor Traceability Rigid Workflow Annoying Notifications FEED ME!!!!! You need to Rebase first. Duh....Sure, sure...I'll wait a couple more weeks Say, when you have a few minutes, could you just take a look at my review? What about my comment on Patch Set 27? When you say "I'll deal with it later", did you mean "never"? Are you really expecting me to review 3k lines of code?!! Uploaded Patch Set 1 No "TODOs"!!! Build Started... Build Failed... Uploaded Patch Set 2 Build Started... Build Unstable... Uploaded Patch Set 3

9 R4E Provides a fully integrated Eclipse peer review environment... My Hero!! Our Hero!!

10 Reviews == Commits Review by Commit or Free form Minimal Structure Stateful Anomalies Anomaly Classes and Ranks Design Rules (Anomaly Patterns) Lack of Visibility across Patch Sets, Reviews Reviews can be grouped by Project, Feature, Team... Anomalies can be Postponed and Tracked across Changes and Reviews Poor Traceability Reviews Items/Artifacts can be assigned to specific Users Time Tracking and Due Date enforcement BIRT Inspection Reports

11 ... Limited Workspace Integration Full JDT/CDT/... integration Code Navigability (across commits, reviews...) Rich Inline Annotations Rigid Workflow Multiple Review Types (Basic, Agile, Formal/IEEE ) Annoying Notifications Integrated s & LDAP support

12 What’s next?: A Vision for Mylyn Reviews Leverage, enhance and extend Gerrit. Support arbitrary review workflows and artifacts with rich Eclipse workspace integration. Provide enterprise review features that go beyond Gerrit and support, efficient, streamlined and collaborative peer reviews. Implement APIs and EMF model-driven tools to support developers for generic review, productivity and task tools. Photo Credit: Hamed Khorramyar

13 Workbench Review Editor Review Editor Review Navigator Task List Task Manager Task Data Task Review Gerrit ? Boooorring!!!

14 Workbench Review Editor Review Editor Review Navigator Task List Task Manager Task Data Task Review Gerrit ? R4E Editor R4E File Store R4E File Store R4E Review R4E Review Navigator

15 Workbench Review Editor Review Editor Review Navigator Task List Task Manager Task Data Task Review Gerrit ? Yawn…

16 Workbench Synchronization/Concurrency Review Editor Review Navigator Task List Task Manager Task Data Task Reviews Review Remote API EMF Task Mapper EMF Task Mapper EMF Edit EMF Edit Gerrit ? Resource Manager Now that’s more interesting…...

17 Workbench Synchronization/Concurrency Task List Task Manager Task Data Task Reviews Review Remote API EMF Task Mapper EMF Task Mapper EMF Edit EMF Edit Gerrit ? Resource Manager Review Navigator Review Editor R4E Editor R4E Navigator R4E Review R4E Gerrit Store R4E File Store R4E File Store

18 Sigh...Not even any pictures in this one.

19 Ah! Now we’re getting somewhere…

20 Ew...Gross! BWWAACCKKK!!!

21 Mylyn project page: Mylyn Reviews wiki page: Mylyn Reviews mailing list: Mylyn Reviews Convergence: R4E sub-subproject home page: R4E User Guide: R4E Blog: R4E Builds: (stable)http://download.eclipse.org/r4e/updates/ (Nightly)http://download.eclipse.org/r4e/snapshots/nightly/ COMMENTS, IDEAS AND CONTRIBUTIONS ARE ALWAYS WELCOME!!!

22 Will there be a version for.NET? Is this going to be on the test? What does "under the supervision of the PMC" mean?

23 1 Sign In: 2 Select Session Evaluate 3 Vote Beyond Gerritblah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah, blah... Loved it!!! +1? I give it a +10! "Approved, submitted, and Merged!"


Download ppt "Miles Parker (Tasktop) Sebastien Dubois (Ericsson) EclipseCon 2013 Hey! What's wrong with Gerrit?!"

Similar presentations


Ads by Google