Merge branch 'develop'

.github/workflows/python-app.yml

@@ -23,7 +23,11 @@ jobs:
     - name: Install dependencies
       run: |
         python -m pip install --upgrade pip
-        pip install tox
-    - name: Lint and test
+        pip install flake8==3.8.4 pytest
+        if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
+    - name: Lint with flake8
       run: |
-        tox
+        python setup.py lint
+    - name: Test with pytest
+      run: |
+        python setup.py test

.gitignore

@@ -9,4 +9,3 @@
 /venv
 build/
 .vscode/settings.json
-.tox/

README.rst

@@ -174,16 +174,13 @@ 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
 ===========
 
-To run tests and the linter run ``pip install tox`` once, then ``tox``.
+To run tests:
+
+``python setup.py test``
+
+To lint:
+
+``python setup.py lint``

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' and el.parent.name != 'pre':
+        if el.parent.name != 'code':
             text = self.escape(text)
 
         # remove trailing whitespaces if any of the following condition is true:

markdownify/main.py

@@ -1,65 +0,0 @@
-#!/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()

setup.cfg

@@ -0,0 +1,2 @@
+[flake8]
+ignore = E501 W503

setup.py

@@ -2,6 +2,7 @@
 import codecs
 import os
 from setuptools import setup, find_packages
+from setuptools.command.test import test as TestCommand, Command
 
 
 read = lambda filepath: codecs.open(filepath, 'r', 'utf-8').read()
@@ -12,7 +13,49 @@ pkgmeta = {
     '__version__': '0.11.2',
 }
 
-read = lambda filepath: codecs.open(filepath, 'r', 'utf-8').read()
+
+class PyTest(TestCommand):
+    def finalize_options(self):
+        TestCommand.finalize_options(self)
+        self.test_args = ['tests', '-s']
+        self.test_suite = True
+
+    def run_tests(self):
+        import pytest
+        errno = pytest.main(self.test_args)
+        raise SystemExit(errno)
+
+
+class LintCommand(Command):
+    """
+    A copy of flake8's Flake8Command
+
+    """
+    description = "Run flake8 on modules registered in setuptools"
+    user_options = []
+
+    def initialize_options(self):
+        pass
+
+    def finalize_options(self):
+        pass
+
+    def distribution_files(self):
+        if self.distribution.packages:
+            for package in self.distribution.packages:
+                yield package.replace(".", os.path.sep)
+
+        if self.distribution.py_modules:
+            for filename in self.distribution.py_modules:
+                yield "%s.py" % filename
+
+    def run(self):
+        from flake8.api.legacy import get_style_guide
+        flake8_style = get_style_guide(config_file='setup.cfg')
+        paths = self.distribution_files()
+        report = flake8_style.check_files(paths)
+        raise SystemExit(report.total_errors > 0)
+
 
 setup(
     name='markdownify',
@@ -26,9 +69,14 @@ setup(
     packages=find_packages(),
     zip_safe=False,
     include_package_data=True,
+    setup_requires=[
+        'flake8>=3.8,<5',
+    ],
+    tests_require=[
+        'pytest>=6.2,<7',
+    ],
     install_requires=[
-        'beautifulsoup4>=4.9,<5',
-        'six>=1.15,<2',
+        'beautifulsoup4>=4.9,<5', 'six>=1.15,<2'
     ],
     classifiers=[
         'Environment :: Web Environment',
@@ -44,9 +92,8 @@ setup(
         'Programming Language :: Python :: 3.8',
         'Topic :: Utilities'
     ],
-    entry_points={
-        'console_scripts': [
-            'markdownify = markdownify.main:main'
-        ]
-    }
+    cmdclass={
+        'test': PyTest,
+        'lint': LintCommand,
+    },
 )

tests/test_conversions.py

@@ -187,7 +187,6 @@ 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():

tox.ini

@@ -1,10 +0,0 @@
-[tox]
-envlist = py38
-
-[testenv]
-deps =
-	flake8
-	pytest
-commands =
-	flake8 --ignore=E501,W503 markdownify tests
-	pytest