- Status:
- Discarded
ExtensionInfo can not parse PKG-INFO with multiline description
Review Request #8001 — Created Feb. 26, 2016 and submitted
ExtensionInfo can not parse PKG-INFO with multiline description
Testing done also with ReviewBoard
Description | From | Last Updated |
---|---|---|
Col: 11 E111 indentation is not a multiple of four |
![]() |
|
Col: 11 E111 indentation is not a multiple of four |
![]() |
|
Col: 11 E111 indentation is not a multiple of four |
![]() |
|
Alphabetize |
|
|
Single quotes on 'PKG-INFO' |
|
|
blank line between these. |
|
|
Blank line between these. |
|
|
This does not handle the case where we cannot decode using any of the encodings. |
|
|
Blank line between these. |
|
|
If data is empty, we don't really need to go through the above, do we? |
|
|
Col: 80 E501 line too long (80 > 79 characters) |
![]() |
|
Col: 80 E501 line too long (81 > 79 characters) |
![]() |
|
This really should be a constant on the class. |
|
|
Can you reflow this? Each sentence needn't take up a whole line. |
|
|
Blank line between these. |
|
|
Single quotes. Can you move the % to the next line? |
|
|
Single quotes. Can you move the % to the next line? |
|
|
Please add a period at the end. |
|
|
It would be a little nicer to write this using for..else (avoiding the "decoded" variable): for enc in self.encodings: try: … |
|
- Status:
- Re-opened
- Testing Done:
-
+ Testing done also with ReviewBoard
- Bugs:
-
- Diff:
Revision 1 (+15 -7)
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: djblets/extensions/extension.py Tool: PEP8 Style Checker Processed Files: djblets/extensions/extension.py
- Change Summary:
-
Introduced fixes for comments and some feedback for still open points
- Diff:
-
Revision 3 (+15 -5)
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: PEP8 Style Checker Processed Files: djblets/extensions/extension.py Tool: Pyflakes Processed Files: djblets/extensions/extension.py
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: djblets/extensions/extension.py Tool: PEP8 Style Checker Processed Files: djblets/extensions/extension.py
- Change Summary:
-
Made encodings as ExtensionInfo class constant
Small style fixes - Diff:
-
Revision 6 (+31 -5)
![reviewbot](https://reviews.reviewboard.org/static/ext/reviewbotext.extension.ReviewBotExtension/images/reviewbot.png)
-
Tool: Pyflakes Processed Files: djblets/extensions/extension.py Tool: PEP8 Style Checker Processed Files: djblets/extensions/extension.py
-
-
-
It would be a little nicer to write this using
for..else
(avoiding the "decoded" variable):for enc in self.encodings: try: data = data.decode(enc) break except UnicodeDecodeError: continue else: logging.warning( 'Failed to decode PKG-INFO content for extension %s', entrypoint.name)
We also don't need to log in the success case.