-
Notifications
You must be signed in to change notification settings - Fork 885
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
pino.transport() #1003
Merged
pino.transport() #1003
Changes from all commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
a2840c8
Basic transport implemented on top of ThreadStream
mcollina b2404c9
Support module transports
mcollina 6b7d7bb
Skip package test on win32
mcollina b3b537a
Ire-enabled failing test on windows
mcollina b3e4178
full cov report
mcollina ec34df7
Revert "full cov report"
mcollina ed8cfd5
ignore lines unreacheable on Windows
mcollina e1c72b9
Update lib/transport.js
mcollina 44c150c
Update lib/transport.js
mcollina 246b6f0
Revert "Revert "full cov report""
mcollina eeca496
try out different ignore strategies
mcollina 661de3e
Revert "Revert "Revert "full cov report"""
mcollina d78a30b
Works on Windows
mcollina c54769e
Emit 'error' if thread exits abruptly
mcollina 2f4170e
Merge branch 'next' into transport
mcollina 4c01c89
Implement pass-through of WorkerThread options
mcollina 463af9d
Esm support with transport
mcollina aa28076
Basic multitransport code
mcollina 2f9649f
multitransport example
mcollina 6dd0627
merged transport and multitransport
mcollina 2ad77ce
Increased timeout
mcollina 465ed17
Added node v16
mcollina 446a581
print the destination
mcollina 8e55539
file://
mcollina 7e8f662
Update example
mcollina cdaa18c
Merge branch 'next' into transport
mcollina 63c4f49
Transports
mcollina 72a9ee0
rm transport local dep from package.json
davidmarkclements b46171d
bump sonic boom
davidmarkclements 153969c
bump pino-std-serializers
davidmarkclements bd28024
transport package test working without local transport dep in package…
davidmarkclements 6ee9e90
typo
davidmarkclements 21abdac
api mods, extra test
davidmarkclements 82578e9
Updated thread-stream
mcollina 8e679ee
fixed node v12 support
mcollina 7e6fb61
Skip failing test on windows
mcollina 47f0df3
print coverage report
mcollina 89fd700
Maybe fix v12
mcollina ad4ee96
print the platform for debugging sake
mcollina bec0c08
Pass windows test
mcollina 35596e6
print coverage data
mcollina ad07d04
Maybe Windows
mcollina 9f1c699
caller refactor
mcollina 97e12a3
Updated API
mcollina f0f06fc
option validation
mcollina 02a4b68
Support modules within destinations
mcollina 58e2914
Use full names for the options
mcollina ca5d4c5
windows coverage
mcollina f879653
coverage report
mcollina a791067
fix coverage on windows
mcollina c924b92
docs first pass
davidmarkclements 52d5e87
target and targets
mcollina 2bbe8c5
restore istanbul ignore
mcollina 2e307a8
Do not check the coverage in CI
mcollina 21f0c40
--no-check-coverage
mcollina 7fe66d6
Merge branch 'next' into transport
mcollina caaec82
removed TODO
mcollina 82ebbbf
built in docs, other doc tweaks
davidmarkclements 2474e29
Update test/transport.test.js
mcollina 6c49f3e
Update test/transport.test.js
mcollina 17dc8a9
Update docs/api.md
mcollina 8ef4409
Update docs/api.md
mcollina 8855a33
Update docs/api.md
mcollina 224474e
Update docs/transports.md
mcollina 55e8b61
Update docs/transports.md
mcollina 45cb21c
PR reviews
mcollina File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a callout here, and in the transports specific docs, to notify the reader about CPU core count. Ideally, a core for the main app + a core per worker thread (transport) should be available.