Skip to content
This repository has been archived by the owner on Nov 14, 2019. It is now read-only.

Allow audience to include multiple mentions, including individual user mentions #158

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

billneff79
Copy link

e.g.

  • audience @user1 @user2 @user3
  • audience @here @userNotHere4
    ...

@codecov-io
Copy link

Codecov Report

Merging #158 into develop will decrease coverage by 1.29%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           develop     #158     +/-   ##
==========================================
- Coverage    86.44%   85.15%   -1.3%     
==========================================
  Files           58       59      +1     
  Lines         1372     1374      +2     
  Branches       181      181             
==========================================
- Hits          1186     1170     -16     
- Misses         186      204     +18
Impacted Files Coverage Δ
lib/bot/setAudience.js 37.5% <0%> (ø) ⬆️
lib/helpers/reports/forChannelUpdate.js 33.33% <0%> (-66.67%) ⬇️
lib/helpers/updateChannelReport.js 19.04% <0%> (-57.15%) ⬇️
lib/helpers/doChannelReport.js 94.73% <0%> (-5.27%) ⬇️
lib/helpers/doBlock.js 91.66% <0%> (-1.67%) ⬇️
models/index.js 92.85% <0%> (ø)
lib/helpers/reports/transformers/index.js 100% <0%> (+7.14%) ⬆️

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 4c8b97a...6af7235. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants