Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork940
Commitfcc7418
committed
Don't explicitly list direct submodules in __all__
This is for non-toplevel __all__, as git.__all__ already did notdo this.As noted in some of the previous commit messags that added them,omitting them might be a bit safer in terms of the impact of bugsbugs in code using GitPython, in that unexpected modules, some ofwhich have the same name as other modules within GitPython, won'tbe imported due to wildcard imports from intermediate subpackages(those that are below the top-level git package/module but collectnon-subpackage modules). Though it is hard to know, since some ofthese would have been imported before, when an __all__ was notdefined at that level.However, a separate benefit of omitting them is consistency withgit.__all__, which does not list the direct Python submodules ofthe git module.This does not affect the output of the modattrs.py script, becausethe attributes exist with the same objects as their values as aresult of those Python submodules being imported (in "from" importsand otherwise), including when importing the top-level git module.1 parent00f4cbc commitfcc7418
File tree
5 files changed
+2
-31
lines changed- git
- index
- objects
- submodule
- refs
- repo
5 files changed
+2
-31
lines changedLines changed: 0 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4 | 4 |
| |
5 | 5 |
| |
6 | 6 |
| |
7 |
| - | |
8 |
| - | |
9 |
| - | |
10 |
| - | |
11 | 7 |
| |
12 | 8 |
| |
13 | 9 |
| |
| |||
16 | 12 |
| |
17 | 13 |
| |
18 | 14 |
| |
19 |
| - | |
20 | 15 |
| |
21 | 16 |
|
Lines changed: 0 additions & 7 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4 | 4 |
| |
5 | 5 |
| |
6 | 6 |
| |
7 |
| - | |
8 |
| - | |
9 |
| - | |
10 |
| - | |
11 |
| - | |
12 |
| - | |
13 | 7 |
| |
14 | 8 |
| |
15 | 9 |
| |
| |||
23 | 17 |
| |
24 | 18 |
| |
25 | 19 |
| |
26 |
| - | |
27 | 20 |
| |
28 | 21 |
| |
29 | 22 |
| |
|
Lines changed: 1 addition & 10 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 | 2 |
| |
3 | 3 |
| |
4 |
| - | |
5 |
| - | |
6 |
| - | |
7 |
| - | |
8 |
| - | |
9 |
| - | |
10 |
| - | |
11 |
| - | |
12 |
| - | |
| 4 | + | |
13 | 5 |
| |
14 |
| - | |
15 | 6 |
| |
16 | 7 |
|
Lines changed: 0 additions & 7 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2 | 2 |
| |
3 | 3 |
| |
4 | 4 |
| |
5 |
| - | |
6 |
| - | |
7 |
| - | |
8 |
| - | |
9 |
| - | |
10 |
| - | |
11 | 5 |
| |
12 | 6 |
| |
13 | 7 |
| |
| |||
19 | 13 |
| |
20 | 14 |
| |
21 | 15 |
| |
22 |
| - | |
23 | 16 |
| |
24 | 17 |
| |
25 | 18 |
| |
|
Lines changed: 1 addition & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
6 |
| - | |
| 6 | + | |
7 | 7 |
| |
8 |
| - | |
9 | 8 |
|
0 commit comments
Comments
(0)