-
-
Notifications
You must be signed in to change notification settings - Fork 99
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rewriting of the manual's chapter 10 (designing packages and classes) #1778
Rewriting of the manual's chapter 10 (designing packages and classes) #1778
Conversation
…ckages As with 4875972 for commands, but this time for raw handlers. It just delegates to the class, but we may want to track these later too, for the same reason as for commands. It also makes things easier for developpers and nicer to document with some symmetry.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see a few things that could be tweaked but I'm too sick to actually do them now and this is a huge and necessary step in the right direction. Merging sooner rather than later ... almost wish we had a HEAD version of the manual being posted on our website so newcomers would be less likely to land on unhelpful docs.
Thanks! I'll go on working on other documentation-related issues progressively---I'd like to reduce the backlog for some of them which seem doable... And at some point maybe we could consider a full re-reading of the manual. There's no urge, though, take care of yourself! |
Back from a few days off away from the internet, here is a proposal to address (a part) of #1465
The very first commit adds
self:registerRawHandler()
to the package API, similar to what was done for commands in 4875972, for the rationale explained in its commit log.The subsequent commits is a rewrite of the manual's "c10" chapter:
pl.class()
constructs, so it's more than time...return
...One thing I have voluntarily removed is all the discussion on "Exports".
I might be wrong, but it should eventually disappear (We can discuss that in a dedicated issue, though.)