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

Make load and start_connectors async #1072

Merged
merged 3 commits into from
Sep 24, 2019

Conversation

jacobtomlinson
Copy link
Member

Description

Make opsdroid.OpsDroid.load and opsdroid.OpsDroid.start_connectors async. This makes it easier to block on connector startup from within start_connectors.

Status

READY

@jacobtomlinson jacobtomlinson mentioned this pull request Sep 23, 2019
4 tasks
@Cadair
Copy link
Contributor

Cadair commented Sep 23, 2019

This looks like it would make things easier.

Copy link
Contributor

@SolarDrew SolarDrew left a comment

Choose a reason for hiding this comment

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

LGTM based on my fairly limited understanding.

opsdroid/__main__.py Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Sep 24, 2019

Codecov Report

Merging #1072 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1072   +/-   ##
=======================================
  Coverage   99.59%   99.59%           
=======================================
  Files          40       40           
  Lines        2450     2450           
=======================================
  Hits         2440     2440           
  Misses         10       10
Impacted Files Coverage Δ
opsdroid/__main__.py 100% <ø> (ø) ⬆️
opsdroid/core.py 100% <100%> (ø) ⬆️

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...2e96a50. Read the comment docs.

@jacobtomlinson jacobtomlinson merged commit 72fc298 into opsdroid:master Sep 24, 2019
@jacobtomlinson jacobtomlinson deleted the async-load branch September 24, 2019 08:48
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

3 participants