Skip to content

Commit

Permalink
Merge pull request #1102 from LeastAuthority/3760.run-codechecks3
Browse files Browse the repository at this point in the history
Try to run codechecks3 on CI

Fixes: ticket:3760
  • Loading branch information
exarkun committed Aug 13, 2021
2 parents 580af66 + 9a8faae commit 561429f
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 8 deletions.
20 changes: 20 additions & 0 deletions .circleci/config.yml
Expand Up @@ -53,6 +53,8 @@ workflows:
# Other assorted tasks and configurations
- "lint":
{}
- "codechecks3":
{}
- "pyinstaller":
{}
- "deprecations":
Expand Down Expand Up @@ -158,6 +160,24 @@ jobs:
command: |
~/.local/bin/tox -e codechecks
codechecks3:
docker:
- <<: *DOCKERHUB_AUTH
image: "circleci/python:3"

steps:
- "checkout"

- run:
name: "Install tox"
command: |
pip install --user tox
- run:
name: "Static-ish code checks"
command: |
~/.local/bin/tox -e codechecks3
pyinstaller:
docker:
- <<: *DOCKERHUB_AUTH
Expand Down
Empty file added newsfragments/3760.minor
Empty file.
9 changes: 1 addition & 8 deletions tox.ini
Expand Up @@ -114,13 +114,6 @@ commands =
[testenv:codechecks]
basepython = python2.7
setenv =
# Workaround an error when towncrier is run under the VCS hook,
# https://stackoverflow.com/a/4027726/624787:
# File "/home/rpatterson/src/work/sfu/tahoe-lafs/.tox/codechecks/lib/python2.7/site-packages/towncrier/check.py", line 44, in __main
# .decode(getattr(sys.stdout, "encoding", "utf8"))
# `TypeError: decode() argument 1 must be string, not None`
PYTHONIOENCODING=utf_8

# If no positional arguments are given, try to run the checks on the
# entire codebase, including various pieces of supporting code.
DEFAULT_FILES=src integration static misc setup.py
Expand Down Expand Up @@ -190,7 +183,7 @@ passenv = TAHOE_LAFS_* PIP_* SUBUNITREPORTER_* USERPROFILE HOMEDRIVE HOMEPATH HO
whitelist_externals =
git
deps =
# see comment in [testenv] about "certifi"
# see comment in [testenv] about "certifi"
certifi
towncrier==21.3.0
commands =
Expand Down

0 comments on commit 561429f

Please sign in to comment.