Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork944
Commit5a8a405
committed
Refactor Git.{AutoInterrupt,CatFileContentStream} nesting
This makes `Git.AutoInterrupt` and `Git.CatFileContentStream`transparent aliases to top-level nonpublic `_AutoInterrupt` and`_CatFileContentStream` classes in the `cmd` module.This does not change the "public" interface. It also does notchange metadata relevant to documentation: the `__name__` and`__qualname__` attributes are set explicitly to the values they hadbefore when these classes were defined nested, so that Sphinxcontinues to document them (and to do so in full) in `Git` and as`Git.AutoInterrupt` and `Git.CatFileContentStream`.The purpose of this is to increase readability. The `Git` class isbig and complex, with a number of long members and various forms ofnesting. Since these two classes can be understood even withoutreading the code of the `Git` class, moving the definitions out ofthe `Git` class into top-level nonpublic classes will hopefullyincrease readability and help with maintenance.1 parentcf8029d commit5a8a405
1 file changed
+226
-214
lines changed0 commit comments
Comments
(0)