Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Add a coveragerc #4180

Merged
merged 5 commits into from
Nov 15, 2018
Merged

Add a coveragerc #4180

merged 5 commits into from
Nov 15, 2018

Conversation

hawkowl
Copy link
Contributor

@hawkowl hawkowl commented Nov 13, 2018

No description provided.

tox.ini Outdated
@@ -122,7 +133,7 @@ skip_install = True
basepython = python3.6
deps =
flake8
commands = /bin/sh -c "flake8 synapse tests scripts scripts-dev scripts/hash_password scripts/register_new_matrix_user scripts/synapse_port_db synctl {env:PEP8SUFFIX:}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is this happening?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

because i mismergedddddd

Copy link
Member

@richvdh richvdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

^

@richvdh
Copy link
Member

richvdh commented Nov 15, 2018

(looks fine to me apart from the apparently unrelated removal of hash_password from the flake8 list, so feel free to merge after fixing that)

@hawkowl hawkowl merged commit a51288e into develop Nov 15, 2018
@hawkowl hawkowl deleted the hawkowl/coveragerc branch November 15, 2018 16:50
anoadragon453 pushed a commit that referenced this pull request Feb 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants