My (limited) experience tells me that if you don't enforce it you
don't get it done
If it's in a profile I can deploy it and then continuum will fail and
somebody will have to chase me and tell me and then I'd have to fix
it, which means that the turnaround is pretty big.
I just added now the duplicate lines thing but I'd like to add also
checkstyle and pmd to enforce some common rules.
On 6/23/06, Brett Porter <[EMAIL PROTECTED]> wrote:
So, you'll fix all duplications when you change the parent?
I still believe this should be in a profile used by CI. The standard
build doesn't need to worry about this.
- Brett
On 23/06/2006 1:18 AM, Carlos Sanchez wrote:
> That's the point, don't accept files with duplications. Better enforce
> it proactively.
>
> On 6/22/06, Brett Porter <[EMAIL PROTECTED]> wrote:
>> On 22/06/2006 10:27 AM, [EMAIL PROTECTED] wrote:
>> > Author: carlos
>> > Date: Wed Jun 21 17:27:49 2006
>> > New Revision: 416184
>> >
>> > URL: http://svn.apache.org/viewvc?rev=416184&view=rev
>> > Log:
>> > Added checkstyle configuration and pmd check for duplications
>>
>> Won't that fail the build?
>>
>> Shouldn't the checks be in a profile?
>>
>> - Brett
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>
>
--
Brett Porter <[EMAIL PROTECTED]>
Apache Maven - http://maven.apache.org/
Better Builds with Maven - http://library.mergere.com/
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
I could give you my word as a Spaniard.
No good. I've known too many Spaniards.
-- The Princess Bride
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
