Update pythonpackage.yml (#49)

* Update pythonpackage.yml

* add requirements for testing

* Update CI / lint comments

* yaml formatting

Author: mattlisiv

.github/workflows/pythonpackage.yml

@@ -0,0 +1,31 @@
+name: Python package
+
+on: [push]
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+    strategy:
+      max-parallel: 4
+      matrix:
+        python-version: [2.7, 3.5, 3.6, 3.7]
+
+    steps:
+    - uses: actions/checkout@v1
+    - name: Set up Python ${{ matrix.python-version }}
+      uses: actions/setup-python@v1
+      with:
+        python-version: ${{ matrix.python-version }}
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements.txt
+    - name: Lint with flake8
+      run: |
+        python -m pip install flake8
+        flake8 . --statistics
+    - name: Test with pytest
+      run: |
+        python -m pip install pytest
+        pytest

newsapi/newsapi_client.py

@@ -14,7 +14,7 @@ def __init__(self, api_key, session=None):
         else:
             self.request_method = session
 
-    def get_top_headlines(
+    def get_top_headlines(  # noqa: C901
         self, q=None, sources=None, language="en", country=None, category=None, page_size=None, page=None
     ):
         """
@@ -132,7 +132,7 @@ def get_top_headlines(
 
         return r.json()
 
-    def get_everything(
+    def get_everything(  # noqa: C901
         self,
         q=None,
         sources=None,
@@ -273,7 +273,7 @@ def get_everything(
 
         return r.json()
 
-    def get_sources(self, category=None, language=None, country=None):
+    def get_sources(self, category=None, language=None, country=None):  # noqa: C901
         """
         Returns the subset of news publishers that top headlines...
 

requirements.txt

@@ -0,0 +1 @@
+requests==2.21.0