dep: Update dependencies (#29)

various update

Author: tachikoma-li

.github/README.md

@@ -47,21 +47,20 @@ git clone [email protected]:qctrl/python-pyquil.git
 
 Once the clone is complete, you have two options:
 
-1. Using setup.py
+1. Using pip
 
    ```shell
    cd python-pyquil
-   python setup.py develop
+   poetry export --dev -f requirements.txt --output requirements.txt --without-hashes
+   pip install -r requirements.txt
+   pip install -e .
    ```
 
-   **Note:** We recommend installing using `develop` to point your installation
-   at the source code in the directory where you cloned the repository.
-
 1. Using Poetry
 
    ```shell
    cd python-pyquil
-   ./setup-poetry.sh
+   poetry install
    ```
 
    **Note:** if you are on Windows, you'll need to install

.github/workflows/on-push.yml

@@ -19,7 +19,7 @@ jobs:
       run: |
         source <(curl -sL http://ci.q-ctrl.com)
         ./ci vault login -r ${{ secrets.VAULT_ROLE_ID }} -s ${{ secrets.VAULT_SECRET_ID }}        
-        ./ci docker run qctrl/python-build:3.7 /scripts/housekeeping.sh
+        ./ci docker run qctrl/ci-images:python-3.7-ci /scripts/housekeeping.sh
 
   linting:
     runs-on: ubuntu-latest
@@ -29,16 +29,16 @@ jobs:
       run: |
         source <(curl -sL http://ci.q-ctrl.com)
         ./ci vault login -r ${{ secrets.VAULT_ROLE_ID }} -s ${{ secrets.VAULT_SECRET_ID }}
-        ./ci docker run qctrl/python-build:3.7 /scripts/install-python-dependencies.sh
+        ./ci docker run qctrl/ci-images:python-3.7-ci /scripts/install-python-dependencies.sh
     - name: Run Pylint
       run: |
-        ./ci docker run qctrl/python-build:3.7 poetry run pylint_runner || true
+        ./ci docker run qctrl/ci-images:python-3.7-ci poetry run pylint_runner || true
     - name: Run Pylama
       run: |
-        ./ci docker run qctrl/python-build:3.7 poetry run pylama || true
+        ./ci docker run qctrl/ci-images:python-3.7-ci poetry run pylama || true
     - name: Run Markdownlint
       run: |
-        ./ci docker run qctrl/python-build:3.7 mdl -- -ig . || true
+        ./ci docker run qctrl/ci-images:python-3.7-ci mdl -- -ig . || true
 
   pytest:
     runs-on: ubuntu-latest
@@ -51,10 +51,10 @@ jobs:
       run: |
         source <(curl -sL http://ci.q-ctrl.com)
         ./ci vault login -r ${{ secrets.VAULT_ROLE_ID }} -s ${{ secrets.VAULT_SECRET_ID }}
-        ./ci docker run qctrl/python-build:${{ matrix.python }} /scripts/install-python-dependencies.sh
+        ./ci docker run qctrl/ci-images:python-${{ matrix.python }}-ci /scripts/install-python-dependencies.sh
     - name: Run Pytest
       run: |
-        ./ci docker run qctrl/python-build:${{ matrix.python }} /scripts/pytest.sh
+        ./ci docker run qctrl/ci-images:python-${{ matrix.python }}-ci /scripts/pytest.sh
 
   publish_internally:
     runs-on: ubuntu-latest
@@ -66,4 +66,4 @@ jobs:
       run: |
         source <(curl -sL http://ci.q-ctrl.com)
         ./ci vault login -r ${{ secrets.VAULT_ROLE_ID }} -s ${{ secrets.VAULT_SECRET_ID }}
-        ./ci docker run qctrl/python-build:3.7 /scripts/publish-dev-version.sh
+        ./ci docker run qctrl/ci-images:python-3.7-ci /scripts/publish-dev-version.sh

.pylintrc

@@ -54,82 +54,23 @@ confidence=
 # --enable=similarities". If you want to run only the classes checker, but have
 # no Warning level messages displayed, use"--disable=all --enable=classes
 # --disable=W"
-disable=duplicate-code,
-        print-statement,
-        parameter-unpacking,
-        unpacking-in-except,
-        old-raise-syntax,
-        backtick,
-        long-suffix,
-        old-ne-operator,
-        old-octal-literal,
-        import-star-module-level,
-        non-ascii-bytes-literal,
-        raw-checker-failed,
+disable=raw-checker-failed,
         bad-inline-option,
         locally-disabled,
-        locally-enabled,
         file-ignored,
         suppressed-message,
         useless-suppression,
         deprecated-pragma,
-        apply-builtin,
-        basestring-builtin,
-        buffer-builtin,
-        cmp-builtin,
-        coerce-builtin,
-        execfile-builtin,
-        file-builtin,
-        long-builtin,
-        raw_input-builtin,
-        reduce-builtin,
-        standarderror-builtin,
-        unicode-builtin,
-        xran
-        ge-builtin,
-        coerce-method,
-        delslice-method,
-        getslice-method,
-        setslice-method,
-        no-absolute-import,
-        old-division,
-        dict-iter-method,
-        dict-view-method,
-        next-method-called,
-        metaclass-assignment,
-        indexing-exception,
-        raising-string,
-        reload-builtin,
-        oct-method,
-        hex-method,
-        nonzero-method,
-        cmp-method,
-        input-builtin,
-        round-builtin,
-        intern-builtin,
-        unichr-builtin,
-        map-builtin-not-iterating,
-        zip-builtin-not-iterating,
-        range-builtin-not-iterating,
-        filter-builtin-not-iterating,
-        using-cmp-argument,
-        eq-without-hash,
-        div-method,
-        idiv-method,
-        rdiv-method,
-        exception-message-attribute,
-        invalid-str-codec,
-        sys-max-int,
-        bad-python3-import,
-        deprecated-string-function,
-        deprecated-str-translate-call,
-        deprecated-itertools-function,
-        deprecated-types-field,
-        next-method-defined,
-        dict-items-not-iterating,
-        dict-keys-not-iterating,
-        dict-values-not-iterating,
-        useless-object-inheritance
+        use-symbolic-message-instead,
+        ## Excessive false positives
+        duplicate-code,
+        ## We have code review to ensure readability, no need for the following
+        ## highly-opinionated checks
+        too-many-arguments,
+        too-many-branches,
+        too-many-lines,
+        too-many-locals,
+        too-many-statements,
 
 # Enable the message, report, category or checker with the given id(s). You can
 # either give multiple identifier separated by comma (,) or put this option
@@ -324,13 +265,6 @@ max-line-length=100
 # Maximum number of lines in a module
 max-module-lines=1000
 
-# List of optional constructs for which whitespace checking is disabled. `dict-
-# separator` is used to allow tabulation in dicts, etc.: {1  : 1,\n222: 2}.
-# `trailing-comma` allows a space between comma and closing bracket: (a, ).
-# `empty-line` allows space-only lines.
-no-space-check=trailing-comma,
-               dict-separator
-
 # Allow the body of a class to be on the same line as the declaration if body
 # contains single statement.
 single-line-class-stmt=no

README.rst

@@ -21,40 +21,44 @@
 
 # -- Path setup --------------------------------------------------------------
 
+import datetime
+
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the
 # documentation root, use os.path.abspath to make it absolute, like shown here.
 #
 import os
 import sys
+
 import toml
 
-sys.path.insert(0, os.path.abspath('..'))
+# pylint:disable=invalid-name
+sys.path.insert(0, os.path.abspath(".."))
 
 # -- Project information -----------------------------------------------------
 parsed = toml.load("../pyproject.toml")
-package_info = parsed['tool']['poetry']
-project = package_info['description']
-author = ", ".join(package_info['authors'])
-release = package_info['version']
-copyright = f"{datetime.datetime.now().year}, Q-CTRL <[email protected]>"
+package_info = parsed["tool"]["poetry"]
+project = package_info["description"]
+author = ", ".join(package_info["authors"])
+release = package_info["version"]
+copyright = f"{datetime.datetime.now().year}, Q-CTRL <[email protected]>"  # pylint: disable=redefined-builtin
 
 # -- General configuration ---------------------------------------------------
 
 # Add any Sphinx extension module names here, as strings. They can be
 # extensions coming with Sphinx (named 'sphinx.ext.*') or your custom
 # ones.
 extensions = [
-    'sphinx.ext.autodoc',
-    'sphinx.ext.coverage',
-    'sphinx.ext.mathjax',
-    'sphinx.ext.napoleon'
+    "sphinx.ext.autodoc",
+    "sphinx.ext.coverage",
+    "sphinx.ext.mathjax",
+    "sphinx.ext.napoleon",
 ]
 
-master_doc = 'index'
+master_doc = "index"
 
 # Add any paths that contain templates here, relative to this directory.
-templates_path = ['_templates']
+templates_path = ["_templates"]
 
 # List of patterns, relative to source directory, that match files and
 # directories to ignore when looking for source files.
@@ -67,12 +71,12 @@
 # The theme to use for HTML and HTML Help pages.  See the documentation for
 # a list of builtin themes.
 #
-html_theme = 'default'
+html_theme = "default"
 
 # Add any paths that contain custom static files (such as style sheets) here,
 # relative to this directory. They are copied after the builtin static files,
 # so a file named "default.css" will overwrite the builtin "default.css".
-html_static_path = ['_static']
+html_static_path = ["_static"]
 html_css_files = [
-    'css/custom.css',
+    "css/custom.css",
 ]