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

insight: add application name to context #419

Merged
merged 1 commit into from Oct 25, 2020
Merged

Conversation

dirkdev98
Copy link
Member

This allows you to put all log streams to a single destination and then aggregate per application.

This allows you to put all log streams to a single destination and then aggregate per application.
@github-actions
Copy link
Contributor

==== Benchmark result ====

Node.js version: 14

Total benchmarks: 9
          Passed: 9
          Failed: 0
-----------
object validator simple      2000000  iterations     504  ns/op
object validator nested        50000  iterations   48183  ns/op
logger - strings              500000  iterations    2413  ns/op
logger - objects              500000  iterations    2706  ns/op
logger - deep objects         500000  iterations    4861  ns/op
uuid                         2000000  iterations     528  ns/op
uuid.isValid                10000000  iterations     166  ns/op
isNil                      500000000  iterations       0  ns/op
isPlainObject              100000000  iterations      18  ns/op

@github-actions
Copy link
Contributor

==== Benchmark result ====

Node.js version: 15

Total benchmarks: 9
          Passed: 9
          Failed: 0
-----------
object validator simple      2000000  iterations     535  ns/op
object validator nested        50000  iterations   47546  ns/op
logger - strings              500000  iterations    2205  ns/op
logger - objects              500000  iterations    2725  ns/op
logger - deep objects         500000  iterations    4991  ns/op
uuid                         2000000  iterations     617  ns/op
uuid.isValid                10000000  iterations     166  ns/op
isNil                      500000000  iterations       0  ns/op
isPlainObject              100000000  iterations      18  ns/op

@github-actions
Copy link
Contributor

==== C8 test coverage ====
Statements: 72.75% ( 17405/23925 )
Branches : 71.28% ( 2045 /2869 )
Functions : 72.03% ( 654 /908 )
Lines : 72.75% ( 17405 /23925 )

@dirkdev98 dirkdev98 merged commit 10d20a1 into main Oct 25, 2020
@dirkdev98 dirkdev98 deleted the feat/insight-application branch October 25, 2020 10:57
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

1 participant