Vincent Siveton wrote:
> 2009/11/6 Dennis Lundberg <[email protected]>:
>> If I understand Vincent's impl the fix disregards any skin and uses the
>> default one. I don't know if that's by design or not.
> 
> The impl was the same as in the past (since mvn 2.0), ie skin is never
> used. I didn't change this logic but it would be an improvement

That's good.

>> I'd like to test drive this release as well, but have been too busy.
>> Will find some time this weekend, if the vote hasn't already passed by then.
> 
> I could wait for you.

I've tried this on the Changes and Checkstyle plugins. On the Changes
plugin I could not see the problem that this issue fixes, but with the
Checkstyle plugin I could see it. Upgrading to 2.0.4.3 solved the
problem in Checkstyle plugin.

So I'm +1 for a release.

> 
> Cheers,
> 
> Vincent
> 
>> Mark Hobson wrote:
>>> Just tried it with maven-checkstyle-plugin:checkstyle.  It produces
>>> the single report, which is an improvement, but the CSSs are missing.
>>> Is this a known issue?
>>>
>>> Cheers,
>>>
>>> Mark
>>>
>>> 2009/11/3 Vincent Siveton <[email protected]>:
>>>> Hi,
>>>>
>>>> This is a maintenance release to fix MSHARED-120.
>>>>
>>>> Staging repo:
>>>> https://repository.apache.org/content/repositories/maven-013/
>>>>
>>>> Staging site:
>>>> http://maven.apache.org/shared/maven-reporting-impl-2.0.4.3
>>>>
>>>> Guide to testing staged releases:
>>>> http://maven.apache.org/guides/development/guide-testing-releases.html
>>>>
>>>> Vote open for 72 hours.
>>>>
>>>> [ ] +1
>>>> [ ] +0
>>>> [ ] -1
>>>>
>>>> ---------------------------------------------------------------------
>>>> To unsubscribe, e-mail: [email protected]
>>>> For additional commands, e-mail: [email protected]
>>>>
>>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [email protected]
>>> For additional commands, e-mail: [email protected]
>>>
>>>
>>
>> --
>> Dennis Lundberg
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
> 
> 


-- 
Dennis Lundberg

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to