Skip to content

Conversation

@Eldinnie
Copy link
Member

Moved tho code to the @message decorator instead

Fixes #752

Moved tho code to the @message decorator instead
@codecov
Copy link

codecov bot commented Sep 10, 2017

Codecov Report

Merging #822 into master will decrease coverage by 1.2%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #822      +/-   ##
==========================================
- Coverage   91.89%   90.69%   -1.21%     
==========================================
  Files         101      101              
  Lines        3987     3985       -2     
  Branches      603      577      -26     
==========================================
- Hits         3664     3614      -50     
- Misses        187      218      +31     
- Partials      136      153      +17
Flag Coverage Δ
#Appveyor ?
#Travis 90.69% <100%> (-0.84%) ⬇️
Impacted Files Coverage Δ
telegram/bot.py 87.43% <100%> (-0.58%) ⬇️
telegram/files/inputfile.py 79.72% <0%> (-10.82%) ⬇️
telegram/base.py 89.74% <0%> (-10.26%) ⬇️
telegram/utils/helpers.py 70% <0%> (-10%) ⬇️
telegram/games/game.py 94.28% <0%> (-5.72%) ⬇️
telegram/ext/messagequeue.py 65.06% <0%> (-4.82%) ⬇️
telegram/utils/webhookhandler.py 92.77% <0%> (-4.82%) ⬇️
telegram/message.py 91.23% <0%> (-4.79%) ⬇️
telegram/utils/request.py 64.28% <0%> (-2.68%) ⬇️
telegram/ext/conversationhandler.py 79.27% <0%> (-1.81%) ⬇️
... and 3 more

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 445bcde...2c4e282. Read the comment docs.

Copy link
Member

@jsmnbom jsmnbom left a comment

Choose a reason for hiding this comment

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

I can't believe we didn't do this sooner xD

@jsmnbom jsmnbom added the 📋 pending-merge work status: pending-merge label Sep 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

📋 pending-merge work status: pending-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@message decorator in Bot

4 participants