Mailing-List: contact flex-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: flex-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of aharui@adobe.com designates
 64.18.1.27 as permitted sender)
From: Alex Harui <aharui@adobe.com>
To: "flex-dev@incubator.apache.org" <flex-dev@incubator.apache.org>
Date: Fri, 22 Jun 2012 11:14:35 -0700
Subject: Re: Fixing bugs and submitting patches (was: pedal down?)
Thread-Topic: Fixing bugs and submitting patches (was: pedal down?)
Thread-Index: Ac1Qic/xgTzDpo+uRe2pYHWd6uwUGgAGRDZq
Message-ID: <CC0A041B.42090%aharui@adobe.com>
In-Reply-To: <39FB646B-063B-43D3-BB82-2A66470B099E@gmail.com>
Accept-Language: en-US
Content-Language: en-US
user-agent: Microsoft-Entourage/13.11.0.110726
acceptlanguage: en-US
Content-Type: text/plain; charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0


On 6/22/12 8:32 AM, "Clint Modien" <cmodien@gmail.com> wrote:

> Great=8A thanks Alex. :)
>=20
> So what's considered a "good" patch?
>=20
> - Fixes the bug?
> - Doesn't break the Mustella tests?
> - Doesn't break the build?
>=20
> Anything else?
I believe this was discussed in the early days related to coding standards.
There is such a long list of things to consider that I think it isn't worth
considering. Fix the bug and submit the patch and depending on who reviews
the patch, you may get asked to make various improvements, but if I'm
reviewing and I get the sense you are going to be active in the project, I'=
m
more likely to let some things go and let you fix them when you become a
committer.

The number 3 reason I reject patches (the number 1 reason is that it doesn'=
t
actually fix the test case, number 2 is it breaks something) is that the fi=
x
only fixes the test case and not the general or underlying problem.

So, if you have a general enough fix and there are no obvious side-effects,
that's usually good enough for me.   If you got most of the coding
conventions right and added comments if needed I'll probably accept it.
Bonus points if you provide an additional mustella or flexunit test for it.
>=20
> Also=8A as far as assigning JIRA tickets to myself=8A should I just leave=
 a
> comment on a ticket saying I'm working on it?
That makes sense to me.
>=20
> Should I check the Adobe ticket for patch files first?
Probably, but there are very few outstanding patches waiting for review.

> These are the types of questions I was hoping to get into the Developer F=
AQ.
>=20
> To update the Developer FAQ should I create a JIRA ticket and submit a pa=
tch
> for the wiki?
Sure.  I haven't been taking the time to deal with web-site and wiki too
much so hopefully some other volunteer will do it.
>=20
>=20

--=20
Alex Harui
Flex SDK Team
Adobe Systems, Inc.
http://blogs.adobe.com/aharui