Skip to content

Commit

Permalink
Add pylint job to Travis CI
Browse files Browse the repository at this point in the history
Now that our logging statements are unified, and that we have a proper
pylint configuration that can check this, we can add a CI pipeline job
that performs the actual check, making non-conforming PRs fail when
logging messages do not adhere to the required convention.

Signed-off-by: Rodrigo Tobar <rtobar@icrar.org>
  • Loading branch information
rtobar committed Jun 14, 2022
1 parent 264758c commit 675d148
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,15 @@ jobs:
script:
- READTHEDOCS=True make -C docs html SPHINXOPTS="-W --keep-going"

# Run pylint to enforce our (few) rules
- name: pylint
python: "3.9"
before_install:
install:
- pip install pylint
script:
- pylint daliuge-common daliuge-translator daliuge-engine


# We want to use docker during the tests
services:
Expand Down

0 comments on commit 675d148

Please sign in to comment.