Code Review Policies
phidias51 at gmail.com
Thu Nov 7 16:08:33 PST 2013
Sorry, I didn't understand the connection between code review and external
hosting. You currently use JIRA (which is hosted on an Oracle server -- at
least that's what whois says), so adding Crucible should just be a matter
of sending email to your infrastructure people shouldn't it?
On Thu, Nov 7, 2013 at 3:54 PM, Richard Bair <richard.bair at oracle.com>wrote:
> Yes, we've been looking at it. But until OpenJDK manages to get some code
> review tool hosted externally, we want something everybody can use.
> On Nov 7, 2013, at 3:53 PM, Mark Fortner <phidias51 at gmail.com> wrote:
> Did you guys ever take a look at Crucible (part of the Atlassian suite)?
> It makes diff's easier to read, and lets you provide feedback in the
> context of the code.
> On Thu, Nov 7, 2013 at 3:21 PM, Richard Bair <richard.bair at oracle.com>wrote:
>> Awesome! Thanks guys. I hope everybody else sees what I see here -- a
>> constant continually effort to improve OpenJFX and make it a real Open
>> Source project in every sense of the word. Major thanks to Steve for
>> pushing on this so hard.
>> On Nov 7, 2013, at 6:36 AM, Stephen F Northover <
>> steve.x.northover at oracle.com> wrote:
>> > Hello Committers,
>> > Let me summarize how to initiate a code review, since this changed
>> > All information about how a bug was fixed needs to be in the JIRA. This
>> means that all patches, webrevs, discussions and who is doing the review
>> needs to be captured there. The email to openjfx-dev is intended to inform
>> the community that a review is happening so others can join in, but it
>> doesn't need to contain detailed information about the fix. People can get
>> all that from the JIRA.
>> > This about it this way: What we are trying to avoid is having any
>> interesting information about the fix appear only in the mailing list. The
>> bottom line is that the comment section of JIRA should contains the
>> contents of the email that previously you would have sent to the list. If
>> you want the information to be in two places, that is fine, but it must be
>> in the JIRA. However, the discussion and any subsequent action is in the
>> > https://wiki.openjdk.java.net/display/OpenJFX/Code+Reviews
>> > Thanks,
>> > Steve and Daniel
More information about the openjfx-dev