remaining AST Browser defects in 1.7

View: New views
6 Messages — Rating Filter:   Alert me  

remaining AST Browser defects in 1.7

by HamletDRC :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Roshan (and others),

Great addition to the AST Browser, the option looks nice. We'll have
to update the "new in 1.7" wiki page with a new screenshot.

There are a few AST Browser defects remaining, but the only one that
should block the release is 3833
(http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
doesn't see any of the JARs loaded in GroovyConsole. My schedule is
busy again so I won't be able to look at this soon.

(And I think Guillaume is the final decider on what is and is not a blocker).

--
Hamlet D'Arcy
hamletdrc@...

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


--
Hamlet D'Arcy

Re: remaining AST Browser defects in 1.7

by Roshan Dawrani-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,
Can someone please confirm if the "new in 1.7" wiki page is http://docs.codehaus.org/display/GROOVY/(draft)+Groovy+1.7+release?

rgds,
Roshan

On Tue, Oct 27, 2009 at 11:33 PM, Hamlet D'Arcy <hamletdrc@...> wrote:
Roshan (and others),

Great addition to the AST Browser, the option looks nice. We'll have
to update the "new in 1.7" wiki page with a new screenshot.

There are a few AST Browser defects remaining, but the only one that
should block the release is 3833
(http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
doesn't see any of the JARs loaded in GroovyConsole. My schedule is
busy again so I won't be able to look at this soon.

(And I think Guillaume is the final decider on what is and is not a blocker).

--
Hamlet D'Arcy
hamletdrc@...

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email




Re: remaining AST Browser defects in 1.7

by HamletDRC :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

> Can someone please confirm if the "new in 1.7" wiki page is
> http://docs.codehaus.org/display/GROOVY/(draft)+Groovy+1.7+release?

that is the page Guillaume has been updating the past few months.


--
Hamlet D'Arcy
hamletdrc@...



On Tue, Oct 27, 2009 at 8:46 PM, Roshan Dawrani
<roshandawrani@...> wrote:

> Hi,
> Can someone please confirm if the "new in 1.7" wiki page is
> http://docs.codehaus.org/display/GROOVY/(draft)+Groovy+1.7+release?
>
> rgds,
> Roshan
>
> On Tue, Oct 27, 2009 at 11:33 PM, Hamlet D'Arcy <hamletdrc@...> wrote:
>>
>> Roshan (and others),
>>
>> Great addition to the AST Browser, the option looks nice. We'll have
>> to update the "new in 1.7" wiki page with a new screenshot.
>>
>> There are a few AST Browser defects remaining, but the only one that
>> should block the release is 3833
>> (http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
>> doesn't see any of the JARs loaded in GroovyConsole. My schedule is
>> busy again so I won't be able to look at this soon.
>>
>> (And I think Guillaume is the final decider on what is and is not a
>> blocker).
>>
>> --
>> Hamlet D'Arcy
>> hamletdrc@...
>>
>> ---------------------------------------------------------------------
>> To unsubscribe from this list, please visit:
>>
>>    http://xircles.codehaus.org/manage_email
>>
>>
>
>

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


--
Hamlet D'Arcy

Re: remaining AST Browser defects in 1.7

by Roshan Dawrani-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,
I have updated the AstBrowser screenshot in "new in 1.7" wiki page.

The screenshot now has a Windows look and feel because that's what I use (I want to feel guilty about it but I really don't :-) )

If it is not fancy enough, someone please take a better picture of it and update it again.

Thanks,
Roshan

On Wed, Oct 28, 2009 at 7:28 AM, Hamlet D'Arcy <hamletdrc@...> wrote:
> Can someone please confirm if the "new in 1.7" wiki page is
> http://docs.codehaus.org/display/GROOVY/(draft)+Groovy+1.7+release?

that is the page Guillaume has been updating the past few months.


--
Hamlet D'Arcy
hamletdrc@...



On Tue, Oct 27, 2009 at 8:46 PM, Roshan Dawrani
<roshandawrani@...> wrote:
> Hi,
> Can someone please confirm if the "new in 1.7" wiki page is
> http://docs.codehaus.org/display/GROOVY/(draft)+Groovy+1.7+release?
>
> rgds,
> Roshan
>
> On Tue, Oct 27, 2009 at 11:33 PM, Hamlet D'Arcy <hamletdrc@...> wrote:
>>
>> Roshan (and others),
>>
>> Great addition to the AST Browser, the option looks nice. We'll have
>> to update the "new in 1.7" wiki page with a new screenshot.
>>
>> There are a few AST Browser defects remaining, but the only one that
>> should block the release is 3833
>> (http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
>> doesn't see any of the JARs loaded in GroovyConsole. My schedule is
>> busy again so I won't be able to look at this soon.
>>
>> (And I think Guillaume is the final decider on what is and is not a
>> blocker).
>>
>> --
>> Hamlet D'Arcy
>> hamletdrc@...
>>
>> ---------------------------------------------------------------------
>> To unsubscribe from this list, please visit:
>>
>>    http://xircles.codehaus.org/manage_email
>>
>>
>
>

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email




Re: remaining AST Browser defects in 1.7

by Roshan Dawrani-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi Hamlet,
I have attached a patch to GROOVY-3833 for code review.

Let me know if you think it will do. I have tested it from Console UI.

rgds,
Roshan

On Tue, Oct 27, 2009 at 11:33 PM, Hamlet D'Arcy <hamletdrc@...> wrote:
Roshan (and others),

Great addition to the AST Browser, the option looks nice. We'll have
to update the "new in 1.7" wiki page with a new screenshot.

There are a few AST Browser defects remaining, but the only one that
should block the release is 3833
(http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
doesn't see any of the JARs loaded in GroovyConsole. My schedule is
busy again so I won't be able to look at this soon.

(And I think Guillaume is the final decider on what is and is not a blocker).

--
Hamlet D'Arcy
hamletdrc@...

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

   http://xircles.codehaus.org/manage_email




Re: remaining AST Browser defects in 1.7

by HamletDRC :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

> I have attached a patch to GROOVY-3833 for code review.

I attached my comment to the issue... there is just one change I would make.

The screenshot looks great. Don't feel bad about Windows (unless
you're planning a Windows 7 party, then you should feel horribly
shamed)


--
Hamlet D'Arcy
hamletdrc@...



On Tue, Oct 27, 2009 at 11:02 PM, Roshan Dawrani
<roshandawrani@...> wrote:

> Hi Hamlet,
> I have attached a patch to GROOVY-3833 for code review.
>
> Let me know if you think it will do. I have tested it from Console UI.
>
> rgds,
> Roshan
>
> On Tue, Oct 27, 2009 at 11:33 PM, Hamlet D'Arcy <hamletdrc@...> wrote:
>>
>> Roshan (and others),
>>
>> Great addition to the AST Browser, the option looks nice. We'll have
>> to update the "new in 1.7" wiki page with a new screenshot.
>>
>> There are a few AST Browser defects remaining, but the only one that
>> should block the release is 3833
>> (http://jira.codehaus.org/browse/GROOVY-3833)... the AST Browser
>> doesn't see any of the JARs loaded in GroovyConsole. My schedule is
>> busy again so I won't be able to look at this soon.
>>
>> (And I think Guillaume is the final decider on what is and is not a
>> blocker).
>>
>> --
>> Hamlet D'Arcy
>> hamletdrc@...
>>
>> ---------------------------------------------------------------------
>> To unsubscribe from this list, please visit:
>>
>>    http://xircles.codehaus.org/manage_email
>>
>>
>
>

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


--
Hamlet D'Arcy