Fix display of codehilite sections which have been iterated.

Review Request #7983 — Created Feb. 19, 2016 and submitted

Information

Djblets
release-0.10.x

Reviewers

When using iter_markdown_lines on something that contains a codehilite block,
the resulting output would display in a very ugly way (and a way that was not
valid HTML). I've improved the iteration and the style for these so they show
correctly.

  • Looked at a change description in Review Board that added some code blocks.
    Before, they looked very ugly, now they look pretty.
  • Ran unit tests.

Description From Last Updated

Col: 67 E703 statement ends with a semicolon

reviewbotreviewbot

undefined name 'line'

reviewbotreviewbot

Col: 80 E501 line too long (99 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (104 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (97 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (91 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (92 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (94 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (90 > 79 characters)

reviewbotreviewbot

Any chance they don't? Can we assert on this, or conditionally apply the logic, in case things change down the …

chipx86chipx86

Col: 80 E501 line too long (91 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (92 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (94 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (90 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (91 > 79 characters)

reviewbotreviewbot

Let's wrap these so that we don't get Review Bot yelling at us for the rest of time. Annoying as …

chipx86chipx86

Col: 80 E501 line too long (92 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (94 > 79 characters)

reviewbotreviewbot

Col: 80 E501 line too long (90 > 79 characters)

reviewbotreviewbot

Col: 17 E131 continuation line unaligned for hanging indent

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
  2. djblets/markdown/__init__.py (Diff revision 1)
     
     
    Show all issues
    Col: 67
     E703 statement ends with a semicolon
    
  3. djblets/markdown/__init__.py (Diff revision 1)
     
     
    Show all issues
     undefined name 'line'
    
  4. djblets/markdown/__init__.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (99 > 79 characters)
    
  5. djblets/markdown/__init__.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (104 > 79 characters)
    
  6. djblets/markdown/__init__.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (97 > 79 characters)
    
  7. djblets/markdown/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (91 > 79 characters)
    
  8. djblets/markdown/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (92 > 79 characters)
    
  9. djblets/markdown/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (94 > 79 characters)
    
  10. djblets/markdown/tests.py (Diff revision 1)
     
     
    Show all issues
    Col: 80
     E501 line too long (90 > 79 characters)
    
  11. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
  2. djblets/markdown/tests.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (91 > 79 characters)
    
  3. djblets/markdown/tests.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (92 > 79 characters)
    
  4. djblets/markdown/tests.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (94 > 79 characters)
    
  5. djblets/markdown/tests.py (Diff revision 2)
     
     
    Show all issues
    Col: 80
     E501 line too long (90 > 79 characters)
    
  6. 
      
chipx86
  1. 
      
  2. djblets/markdown/__init__.py (Diff revision 2)
     
     
     
     
     
     
     
    Show all issues

    Any chance they don't? Can we assert on this, or conditionally apply the logic, in case things change down the road?

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
  2. djblets/markdown/tests.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (91 > 79 characters)
    
  3. djblets/markdown/tests.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (92 > 79 characters)
    
  4. djblets/markdown/tests.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (94 > 79 characters)
    
  5. djblets/markdown/tests.py (Diff revision 3)
     
     
    Show all issues
    Col: 80
     E501 line too long (90 > 79 characters)
    
  6. 
      
chipx86
  1. 
      
  2. djblets/markdown/tests.py (Diff revision 3)
     
     
     
     
     
    Show all issues

    Let's wrap these so that we don't get Review Bot yelling at us for the rest of time. Annoying as that formatting may be.

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
  2. djblets/markdown/tests.py (Diff revision 4)
     
     
    Show all issues
    Col: 17
     E131 continuation line unaligned for hanging indent
    
    1. Arrrrr.

    2. Sometimes I think I like the idea of Slack reaction emojis, for this very reason.

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/markdown/__init__.py
        djblets/markdown/tests.py
    
    Ignored Files:
        djblets/static/djblets/css/mixins/markdown.less
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.10.x (27deb62)
Loading...