Merge pull request #29 from Machine-Learning-for-Medical-Language/mikix/pep8

lint: run pycodestyle too

Author: mikix

.github/workflows/ci.yaml

@@ -35,8 +35,13 @@ jobs:
       - name: Install linters
         run: |
           python -m pip install --upgrade pip
-          pip install pylint
+          pip install pycodestyle pylint
+
+      - name: Run pycodestyle
+        run: |
+          pycodestyle --max-line-length=120 .
 
       - name: Run pylint
+        if: success() || failure() # still run pylint if pycodestyle fails
         run: |
           pylint ctakesclient/ scripts/* test/

test/integration/test_client_covid_symptoms.py

@@ -25,8 +25,8 @@ def test_covid_symptoms_medical_synonyms(self):
             'Myalgias': 'Muscle aches and pain',
             'Chills': 'Fever or chills',
             'Post-tussive': 'after Coughing',
-            'tussive':'related to Coughing',
-            'Pharyngitis':'sore throat',
+            'tussive': 'related to Coughing',
+            'Pharyngitis': 'sore throat',
             'Odynophagia': 'sore throat',
             'Loss of taste': 'Anosmia',
             'Loss of smell': 'Anosmia',

test/integration/test_client_ctakes_rest_server.py

@@ -4,6 +4,7 @@
 import ctakesclient
 from test.test_resources import LoadResource
 
+
 class TestClientCtakesRestServer(unittest.TestCase):
     """Test case for REST requests"""
 

test/integration/test_negation_transformer.py

@@ -96,5 +96,6 @@ def assertPolarityCompatible(self, text: str):
 
         self.assertEqual(polarities_ctakes, polarities_cnlp, text)
 
+
 if __name__ == '__main__':
     unittest.main()