Repository: matthewwithanm/python-markdownify
Pull request #67 (2022-05-04 10:16:37)

don't escape text in pre tag (Fenced Code Blocks)

AlexVonB: Hi! Thanks for the PR, please add some tests to

https://github.com/matthewwithanm/python-markdownify/blob/5adda130b8424bad9d0ddb5a2cd7e9fa74e817fb/tests/test_conversions.py#L187-L189

to make this feature future proof. Thanks a lot!
tjmnmk: Hi, sorry, I've added it.

Patch:
diff --git a/README.rst b/README.rst
index 5b0d530..beb7f8b 100644
--- a/README.rst
+++ b/README.rst
@@ -174,6 +174,15 @@ change:
         return ImageBlockConverter(**options).convert(html)
 
 
+Command Line Interface
+=====================
+
+Use ``markdownify example.html > example.md`` or pipe input from stdin
+(``cat example.html | markdownify > example.md``).
+Call ``markdownify -h`` to see all available options.
+They are the same as listed above and take the same arguments.
+
+
 Development
 ===========
 
diff --git a/markdownify/__init__.py b/markdownify/__init__.py
index a49eed4..e15ecd4 100644
--- a/markdownify/__init__.py
+++ b/markdownify/__init__.py
@@ -158,7 +158,7 @@ class MarkdownConverter(object):
                     and el.parent.parent.name == 'pre')):
             text = whitespace_re.sub(' ', text)
 
-        if el.parent.name != 'code':
+        if el.parent.name != 'code' and el.parent.name != 'pre':
             text = self.escape(text)
 
         # remove trailing whitespaces if any of the following condition is true:
diff --git a/markdownify/main.py b/markdownify/main.py
new file mode 100644
index 0000000..87c7c77
--- /dev/null
+++ b/markdownify/main.py
@@ -0,0 +1,65 @@
+#!/usr/bin/env python
+
+import argparse
+import sys
+
+from markdownify import markdownify
+
+
+def main(argv=sys.argv[1:]):
+    parser = argparse.ArgumentParser(
+        prog='markdownify',
+        description='Converts html to markdown.',
+    )
+
+    parser.add_argument('html', nargs='?', type=argparse.FileType('r'),
+                        default=sys.stdin,
+                        help="The html file to convert. Defaults to STDIN if not "
+                        "provided.")
+    parser.add_argument('-s', '--strip', nargs='*',
+                        help="A list of tags to strip. This option can't be used with "
+                        "the --convert option.")
+    parser.add_argument('-c', '--convert', nargs='*',
+                        help="A list of tags to convert. This option can't be used with "
+                        "the --strip option.")
+    parser.add_argument('-a', '--autolinks', action='store_true',
+                        help="A boolean indicating whether the 'automatic link' style "
+                        "should be used when a 'a' tag's contents match its href.")
+    parser.add_argument('--default-title', action='store_false',
+                        help="A boolean to enable setting the title of a link to its "
+                        "href, if no title is given.")
+    parser.add_argument('--heading-style',
+                        choices=('ATX', 'ATX_CLOSED', 'SETEXT', 'UNDERLINED'),
+                        help="Defines how headings should be converted.")
+    parser.add_argument('-b', '--bullets', default='*+-',
+                        help="A string of bullet styles to use; the bullet will "
+                        "alternate based on nesting level.")
+    parser.add_argument('--sub-symbol', default='',
+                        help="Define the chars that surround '<sub>'.")
+    parser.add_argument('--sup-symbol', default='',
+                        help="Define the chars that surround '<sup>'.")
+    parser.add_argument('--code-language', default='',
+                        help="Defines the language that should be assumed for all "
+                        "'<pre>' sections.")
+    parser.add_argument('--no-escape-asterisks', dest='escape_asterisks',
+                        action='store_false',
+                        help="Do not escape '*' to '\\*' in text.")
+    parser.add_argument('--no-escape-underscores', dest='escape_underscores',
+                        action='store_false',
+                        help="Do not escape '_' to '\\_' in text.")
+    parser.add_argument('-i', '--keep-inline-images-in', nargs='*',
+                        help="Images are converted to their alt-text when the images are "
+                        "located inside headlines or table cells. If some inline images "
+                        "should be converted to markdown images instead, this option can "
+                        "be set to a list of parent tags that should be allowed to "
+                        "contain inline images.")
+    parser.add_argument('-w', '--wrap', action='store_true',
+                        help="Wrap all text paragraphs at --wrap-width characters.")
+    parser.add_argument('--wrap-width', type=int, default=80)
+
+    args = parser.parse_args(argv)
+    print(markdownify(**vars(args)))
+
+
+if __name__ == '__main__':
+    main()
diff --git a/setup.py b/setup.py
index dd7c8b1..171c45d 100644
--- a/setup.py
+++ b/setup.py
@@ -96,4 +96,9 @@ setup(
         'test': PyTest,
         'lint': LintCommand,
     },
+    entry_points={
+        'console_scripts': [
+            'markdownify = markdownify.main:main'
+        ]
+    }
 )


Test patch:
diff --git a/tests/test_conversions.py b/tests/test_conversions.py
index 04c8db7..da78649 100644
--- a/tests/test_conversions.py
+++ b/tests/test_conversions.py
@@ -187,6 +187,7 @@ def test_p():
 def test_pre():
     assert md('<pre>test\n    foo\nbar</pre>') == '\n```\ntest\n    foo\nbar\n```\n'
     assert md('<pre><code>test\n    foo\nbar</code></pre>') == '\n```\ntest\n    foo\nbar\n```\n'
+    assert md('<pre>this_should_not_escape</pre>') == '\n```\nthis_should_not_escape\n```\n'
 
 
 def test_s():