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.

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

Remove queue-flow, migrate two files to ES6 #11

Merged
merged 3 commits into from
Aug 29, 2018
Merged

Conversation

dfellis
Copy link
Owner

@dfellis dfellis commented Aug 29, 2018

No description provided.

@dfellis dfellis self-assigned this Aug 29, 2018
@coveralls
Copy link

coveralls commented Aug 29, 2018

Pull Request Test Coverage Report for Build 34

  • 491 of 538 (91.26%) changed or added relevant lines in 11 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+3.8%) to 88.326%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/transports/server/http.js 24 25 96.0%
lib/transports/server/tcp.js 53 54 98.15%
lib/transports/server/childProcess.js 27 30 90.0%
lib/transports/client/tcp.js 112 115 97.39%
lib/transports/server/middleware.js 16 20 80.0%
lib/transports/shared/tcp.js 54 59 91.53%
lib/transports/client/childProcess.js 58 64 90.63%
lib/server.js 62 86 72.09%
Totals Coverage Status
Change from base Build 28: 3.8%
Covered Lines: 518
Relevant Lines: 565

💛 - Coveralls

@dfellis dfellis merged commit f09f0c6 into master Aug 29, 2018
@dfellis dfellis deleted the remove-queue-flow branch August 29, 2018 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants