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.

Sign up for GitHub

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

Jump to bottom

100% coverage #45

Closed
sindresorhus opened this issue Oct 1, 2018 · 4 comments · Fixed by #53
Closed

100% coverage #45

sindresorhus opened this issue Oct 1, 2018 · 4 comments · Fixed by #53
Labels
enhancement 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt help wanted

Comments

@sindresorhus
Copy link
Owner

sindresorhus commented Oct 1, 2018 •
edited by issuehunt-oss bot
Loading

Issuehunt badges

We're currently at 90%. Would be great to get to 100% to ensure we don't introduce regressions in untested code paths.

https://codecov.io/gh/sindresorhus/p-queue/src/master/index.js

ameech earned $20.00 by resolving this issue!
@dalefrancis88

This comment has been minimized.

Sign in to view
@IssueHuntBot
Copy link

@issuehuntfest has funded $20.00 to this issue. See it on IssueHunt

@IssueHuntBot
Copy link

@ameech has submitted a pull request. See it on IssueHunt

@ameech ameech mentioned this issue Jan 17, 2019
Merged
sindresorhus pushed a commit that referenced this issue Mar 12, 2019
@ameech @sindresorhus
100% code coverage (#53)
37c1204 
Fixes #45
@IssueHuntBot
Copy link

@sindresorhus has rewarded $18.00 to @ameech. See it on IssueHunt

  • 💰 Total deposit: $20.00
  • 🎉 Repository reward(0%): $0.00
  • 🔧 Service fee(10%): $2.00

@issuehunt-oss issuehunt-oss bot added the 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt label May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement 🎁 Rewarded on Issuehunt This issue has been rewarded on Issuehunt help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update code coverage
3 participants
@sindresorhus @dalefrancis88 @IssueHuntBot