Skip to content

Conversation

vincehi
Copy link

@vincehi vincehi commented Sep 11, 2025

Update transports.md with pino-email-transport

Update transports.md witj pino-email-transport
Copy link
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the new parts in alphabetical order. Also, we usually like to see that the modules added to the list have tests.

@vincehi
Copy link
Author

vincehi commented Sep 12, 2025

Please add the new parts in alphabetical order. Also, we usually like to see that the modules added to the list have tests.

I sorted them alphabetically. I will write the tests when I have some time. I'll add it to my TODO list :)

@jsumners jsumners requested a review from mcollina September 18, 2025 10:34
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would need to be solved vincehi/pino-email-transport#1

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.

3 participants