Helpme

[vinicius0197]

New Feature Submissions:

To run tests locally, use make if you're on Mac or Linux.

1. [ ] Does your submission pass all the tests?
2. [ ] Have you lint your code locally prior to submission?
3. [ ] Have you written proper tests to your added code?

New Dialog Submissions:

User messages have - -, bot messages have -

1. [ ] Does all your user messages have lower letters?
2. [ ] Have you lint your dialogs locally prior to submission?
3. [ ] Have you written proper tests to your added dialogs on test_dialogs?
4. [ ] Does your dialog have an existing category? If not, did you create it?

Changes to Core Features:

• [ ] Have you added an explanation of what your changes do and why you'd like us to include them?
• [ ] Have you written new tests or modified old ones for your core changes, as applicable?
• [ ] Have you successfully ran tests with your changes locally?

[codeclimate[bot]]

Code Climate has analyzed commit 0857ad66 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Style	2

The test coverage on the diff in this pull request is 42.8% (80% is the threshold).

This pull request will bring the total coverage in the repository to 93.1%.

View more on Code Climate.

[vinicius0197]

@alexandrebarbaruiva Concertei os erros do Travis, mas o codeclimate reclamou do tamanho de uma das funções. Seria melhor colocar aquela lista de contatos para fora da função?

E outra coisa, eu classifico esse PR como New Feature ou New Dialog?

[codecov[bot]]

Codecov Report

Merging #37 into master will decrease coverage by 5.26%. The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
- Coverage     100%   94.73%   -5.27%     
==========================================
  Files           2        2              
  Lines         102      114      +12     
  Branches        4        4              
==========================================
+ Hits          102      108       +6     
- Misses          0        6       +6

Impacted Files	Coverage Δ
bot/application.py	92.1% <50%> (-7.9%)	: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 48e0b04...d5d4060. Read the comment docs.

[alexandrebarbaruiva]

@vinicius0197 como new feature!

[codecov[bot]]

Codecov Report

Merging #37 into master will decrease coverage by 6.89%. The diff coverage is 42.85%.

@@           Coverage Diff            @@
##           master     #37     +/-   ##
========================================
- Coverage     100%   93.1%   -6.9%     
========================================
  Files           2       2             
  Lines         102     116     +14     
  Branches        4       4             
========================================
+ Hits          102     108      +6     
- Misses          0       8      +8

Impacted Files	Coverage Δ
bot/application.py	89.74% <42.85%> (-10.26%)	: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 1be708d...0857ad6. Read the comment docs.