Tweak pre commit and re-enable CCI PR tests

Author: ns-axu

.circleci/config.yml

@@ -3,7 +3,35 @@ version: 2.1
 orbs:
   ghpr: narrativescience/[email protected]
 
+commands:
+  install-deps:
+    description: Install dependencies and initialize pre-commit hooks
+    steps:
+      - run:
+          command: |
+            pip install pre-commit tox flit
+            pre-commit install
+          name: Install dependencies
+
 jobs:
+  test:
+    docker:
+      - image: circleci/python:3.7
+    steps:
+      - ghpr/build-prospective-branch
+      - install-deps
+      - run:
+          name: Run commit hooks
+          command: |
+            pre-commit run \
+              --source "origin/${GITHUB_PR_BASE_BRANCH}" \
+              --origin "origin/${CIRCLE_BRANCH}" \
+              --show-diff-on-failure
+      - run: tox
+      - run: flit build
+      - ghpr/post-pr-comment:
+          comment: Tests failed!
+          when: on_fail
   publish:
     docker:
       - image: circleci/python:3.7
@@ -14,6 +42,13 @@ jobs:
       - run: flit publish
 
 workflows:
+  pull_request:
+    jobs:
+      - test:
+          filters:
+            branches:
+              ignore:
+                - master
   publish:
     jobs:
       - publish:

.flake8

@@ -40,20 +40,3 @@ repos:
         language: python
         types: [file, python]
         additional_dependencies: [black==19.3b0]
-
-      - id: pydocstyle
-        name: Lint Python docstrings (pydocstyle)
-        entry: pydocstyle
-        language: python
-        types: [file, python]
-        additional_dependencies: [pydocstyle==4.0.1]
-
-      - id: flake8
-        name: Lint Python (flake8)
-        entry: flake8 --config .flake8
-        language: python
-        types: [file, python]
-        additional_dependencies:
-          - flake8==3.7.9
-          - "flake8-import-order<0.19,>=0.18"
-          - flake8-print>=3.1.4,<4

.pre-commit-config.yaml

@@ -1,3 +1,4 @@
+aiohttp >= 3.6.2
 certifi >= 14.05.14
 future; python_version<="2.7"
 six >= 1.10