search

ovhemert / pino-applicationinsights

A transport for pino that sends messages to Azure Application Insights
MIT License
10 stars 12 forks source link

breaking-createWriteStreamSync-simplest-src-changes #59

Closed devinrhode2 closed 2 years ago

devinrhode2 commented 2 years ago

this PR page seems to have stale cache: https://github.com/ovhemert/pino-applicationinsights/pull/58

simply remove async/await/Promise constructs from src

no test changes no refactoring no cleanup just remove all async/await stuff

This is an incomplete PR. I created it to use as a patch via patch-package. I am opening this PR simply as a way to make it easier to understand my other PR "Convert createWriteStream to createWriteStreamSync", which includes updated tests, docs, etc.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

devinrhode2 commented 2 years ago

Hoarding

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.