Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Re-enable pylint in CI #3293

Merged
merged 13 commits into from
Nov 11, 2019
Merged

Re-enable pylint in CI #3293

merged 13 commits into from
Nov 11, 2019

Commits on Nov 4, 2019

  1. Revert "Remove linting from 4.1 release" 

    This reverts commit f2ea574.
    @jngrad
    jngrad committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    a2a779e View commit details
    Browse the repository at this point in the history

  2. Use a .pylintrc file

    @jngrad
    jngrad committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    2ec901e View commit details
    Browse the repository at this point in the history

  3. espresso-ci: post pylint warnings to the PR 

    Script `maintainer/gh_post_pylint.py` is derived from
script `maintainer/gh_post_docs_warnings.py`.
    @jngrad
    jngrad committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    03ccea7 View commit details
    Browse the repository at this point in the history

Commits on Nov 5, 2019

  1. Move bash code from YAML to fix_style.sh

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    8b42199 View commit details
    Browse the repository at this point in the history

  2. Add pylint pragmas

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    f420afc View commit details
    Browse the repository at this point in the history

  3. Remove unused imports

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    d649d99 View commit details
    Browse the repository at this point in the history

  4. Remove unused variables, arguments, functions

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    38a9200 View commit details
    Browse the repository at this point in the history

  5. Fix overshadowed test

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    faf57c5 View commit details
    Browse the repository at this point in the history

  6. Update code and tutorials

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    9662cb9 View commit details
    Browse the repository at this point in the history

  7. Use a recent pylint version 

    These version numbers are the default in Ubuntu 19.10.
    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    eab12d1 View commit details
    Browse the repository at this point in the history

  8. Add guards 

    Make sure espresso-ci deletes the correct message using well-defined
tokens that are guanranteed to be in the posted message. Don't
silently ignore Python exceptions.
    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    579c68f View commit details
    Browse the repository at this point in the history

  9. Change status message

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    04872c2 View commit details
    Browse the repository at this point in the history

  10. Doxygen style check: update to Ubuntu 18.04

    @jngrad
    jngrad committed Nov 5, 2019
    Configuration menu
    Copy the full SHA
    d8418d8 View commit details
    Browse the repository at this point in the history