busy-beaver-dev / busy-beaver

The Chicago Python Community Engagement Slack bot
https://busybeaverbot.com
MIT License
70 stars 23 forks source link

Run post_github_summary_message coroutine via RQ #359

Open alysivji opened 3 years ago

alysivji commented 3 years ago

What does this do

Why are we doing this

Don't have to manually queue up coroutines via asyncio. Results in a cleaner API

How should this be tested

Added pytest-asyncio to make it easier to test coroutines

Migrations

n/a

Dependencies

n/a

Callouts

n/a

codecov-commenter commented 3 years ago

Codecov Report

Merging #359 (b7e231f) into master (5e3543f) will decrease coverage by 0.13%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #359      +/-   ##
==========================================
- Coverage   89.51%   89.38%   -0.14%     
==========================================
  Files          71       71              
  Lines        2509     2506       -3     
==========================================
- Hits         2246     2240       -6     
- Misses        263      266       +3     
Impacted Files Coverage Δ
busy_beaver/apps/debug/workflows.py 0.00% <0.00%> (ø)
...beaver/apps/github_integration/summary/workflow.py 95.45% <100.00%> (ø)
busy_beaver/common/wrappers/github.py 97.08% <100.00%> (-0.17%) :arrow_down:

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 5e3543f...b7e231f. Read the comment docs.