Refactoring HelpTopic (needs review)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

Refactoring HelpTopic (needs review)

Nicolai Hess-3-2
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai
Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

stepharo

I will!

I love to read nice comments

Stef


Le 28/7/16 à 00:38, Nicolai Hess a écrit :
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai

Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

Nicolai Hess-3-2


2016-07-28 16:18 GMT+02:00 stepharo <[hidden email]>:

I will!

I love to read nice comments

Stef



I updated the slice and added some more example (much more is needed but I think it is better than having no doc :) ).
I changed some help topics on refactoring support in nautilus to directly used the refactoring classes comments (no need to maintain two docs), but
this is not applicable for all refactorings, for example "generate superclass" and "insert superclass" needs its own description, as there is only one RBAddClassRefactoring comment.

please help with reviewing, I am not good at writing this kind of text.

The refactoring engine is a great and powerfull piece of code and it would be great to have a proper documentation "in-image".
I have seen some bugs, bad code or wrong bug fixes by contributors not fully understanding the framework (me too :) ).

just load the slice and open
HelpBrowser openOn: RefactoringHelp.


thanks
nicolai


 
Le 28/7/16 à 00:38, Nicolai Hess a écrit :
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai


Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

Nicolai Hess-3-2


2016-08-06 15:24 GMT+02:00 Nicolai Hess <[hidden email]>:


2016-07-28 16:18 GMT+02:00 stepharo <[hidden email]>:

I will!

I love to read nice comments

Stef



I updated the slice and added some more example (much more is needed but I think it is better than having no doc :) ).
I changed some help topics on refactoring support in nautilus to directly used the refactoring classes comments (no need to maintain two docs), but
this is not applicable for all refactorings, for example "generate superclass" and "insert superclass" needs its own description, as there is only one RBAddClassRefactoring comment.

please help with reviewing, I am not good at writing this kind of text.

The refactoring engine is a great and powerfull piece of code and it would be great to have a proper documentation "in-image".
I have seen some bugs, bad code or wrong bug fixes by contributors not fully understanding the framework (me too :) ).

just load the slice and open
HelpBrowser openOn: RefactoringHelp.


thanks
nicolai


I'll put it on fixed to review now,
If you have any comments, notes or suggestions please comment on the issue or
in this thread

thanks
nicolai
 

 
Le 28/7/16 à 00:38, Nicolai Hess a écrit :
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai



Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

stepharo
In reply to this post by Nicolai Hess-3-2

Hi nicolai

I remember that I read everything and fix typos. Did you got my review or did I suuceed to lose it ;).

Stef



Le 6/8/16 à 15:24, Nicolai Hess a écrit :


2016-07-28 16:18 GMT+02:00 stepharo <[hidden email]>:

I will!

I love to read nice comments

Stef



I updated the slice and added some more example (much more is needed but I think it is better than having no doc :) ).
I changed some help topics on refactoring support in nautilus to directly used the refactoring classes comments (no need to maintain two docs), but
this is not applicable for all refactorings, for example "generate superclass" and "insert superclass" needs its own description, as there is only one RBAddClassRefactoring comment.

please help with reviewing, I am not good at writing this kind of text.

The refactoring engine is a great and powerfull piece of code and it would be great to have a proper documentation "in-image".
I have seen some bugs, bad code or wrong bug fixes by contributors not fully understanding the framework (me too :) ).

just load the slice and open
HelpBrowser openOn: RefactoringHelp.


thanks
nicolai


 
Le 28/7/16 à 00:38, Nicolai Hess a écrit :
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai



Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

Nicolai Hess-3-2


2016-08-24 6:49 GMT+02:00 stepharo <[hidden email]>:

Hi nicolai

I remember that I read everything and fix typos. Did you got my review or did I suuceed to lose it ;).


No, I think it was lost :-/
 

Stef


But the helpl is now in (60192), but it is still not finished, only a start, needs more examples and maybe better class comments.
Everyone who wants to help, just try it out

HelpBrowser openOn:RefactoringHelp

read it and give feedback.

If you find any typos, wrong or unclear information just report back, and I'll fix it.

we need more and better docs
we need more and better docs
and it would help if we have
more and better docs :-)


 



Le 6/8/16 à 15:24, Nicolai Hess a écrit :


2016-07-28 16:18 GMT+02:00 stepharo <[hidden email]>:

I will!

I love to read nice comments

Stef



I updated the slice and added some more example (much more is needed but I think it is better than having no doc :) ).
I changed some help topics on refactoring support in nautilus to directly used the refactoring classes comments (no need to maintain two docs), but
this is not applicable for all refactorings, for example "generate superclass" and "insert superclass" needs its own description, as there is only one RBAddClassRefactoring comment.

please help with reviewing, I am not good at writing this kind of text.

The refactoring engine is a great and powerfull piece of code and it would be great to have a proper documentation "in-image".
I have seen some bugs, bad code or wrong bug fixes by contributors not fully understanding the framework (me too :) ).

just load the slice and open
HelpBrowser openOn: RefactoringHelp.


thanks
nicolai


 
Le 28/7/16 à 00:38, Nicolai Hess a écrit :
Hi,

I recently added some class comments to the Refactoring-Package. (RBBrowserEnvironments, RBRefactoring, RBNamespace).

Now I added some more HelpTopics describing the refactoring framework, some background information, RBPatternParser and Metavariables, some examples and put the existing class comments into the HelpBook.

see fogbugz issue 18829 Add a RefactoringFramework System HelpTopic
you can load the slice and open the HelpBook with

HelpBrowser openOn: RefactoringHelp.

Would be nice if someone can help with proofreading and of course, amy feedback is welcome.

nicolai




Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

stepharo



Le 24/8/16 à 11:20, Nicolai Hess a écrit :


2016-08-24 6:49 GMT+02:00 stepharo <[hidden email]>:

Hi nicolai

I remember that I read everything and fix typos. Did you got my review or did I suuceed to lose it ;).


No, I think it was lost :-/

oops did you check if it is not in the inbox repo?
Do you have the name of the package so that I check if I have it on my machine?
 

Stef


But the helpl is now in (60192), but it is still not finished, only a start, needs more examples and maybe better class comments.
Everyone who wants to help, just try it out

HelpBrowser openOn:RefactoringHelp

read it and give feedback.

If you find any typos, wrong or unclear information just report back, and I'll fix it.

we need more and better docs
we need more and better docs
and it would help if we have
more and better docs :-)

:)


Reply | Threaded
Open this post in threaded view
|

Re: Refactoring HelpTopic (needs review)

Nicolai Hess-3-2


2016-08-25 7:13 GMT+02:00 stepharo <[hidden email]>:



Le 24/8/16 à 11:20, Nicolai Hess a écrit :


2016-08-24 6:49 GMT+02:00 stepharo <[hidden email]>:

Hi nicolai

I remember that I read everything and fix typos. Did you got my review or did I suuceed to lose it ;).


No, I think it was lost :-/

oops did you check if it is not in the inbox repo?
Do you have the name of the package so that I check if I have it on my machine?

Nothing in inbox for 6.0, besides my own slices for issue 18829.
The packages are
Refactoring-Help and Refactoring-Core

 
 

Stef


But the helpl is now in (60192), but it is still not finished, only a start, needs more examples and maybe better class comments.
Everyone who wants to help, just try it out

HelpBrowser openOn:RefactoringHelp

read it and give feedback.

If you find any typos, wrong or unclear information just report back, and I'll fix it.

we need more and better docs
we need more and better docs
and it would help if we have
more and better docs :-)

:)