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.

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

Add flake8 to the precommit hooks #1073

Merged
merged 1 commit into from
Sep 24, 2019

Conversation

jacobtomlinson
Copy link
Member

Description

Add flake8 to the precommit hooks

@codecov
Copy link

codecov bot commented Sep 24, 2019

Codecov Report

Merging #1073 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1073   +/-   ##
=======================================
  Coverage   99.59%   99.59%           
=======================================
  Files          40       40           
  Lines        2450     2450           
=======================================
  Hits         2440     2440           
  Misses         10       10

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e09060b...e72a935. Read the comment docs.

@jacobtomlinson jacobtomlinson merged commit 4e2f642 into opsdroid:master Sep 24, 2019
@jacobtomlinson jacobtomlinson deleted the precommit-flake8 branch September 24, 2019 08:42
jacobtomlinson pushed a commit that referenced this pull request Sep 24, 2019
* Updating readme with docker swarm/compose example

* Adding openssh-client to dockerfile, adding key_path as a skill parameter

* test_install_specific_remote_module required None type on git_clone call

* test attempt

* reworking

* adding None type to test_instalL-default_remote_module

* running black

* Update aiohttp from 3.6.0 to 3.6.1 (#1066)

* Update webexteamssdk from 1.1.1 to 1.2 (#1067)

* Add flake8 to the precommit hooks (#1073)

* Make load and start_connectors async (#1072)

* Make load async

* Add missing docstring

* Undo wrapping config path in string

* Test git clone ssh rsa path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant