googleapis / nodejs-logging-winston

Node.js client integration between Stackdriver Logging and Winston.
https://cloud.google.com/logging/
Apache License 2.0
105 stars 50 forks source link

fix: Fix trace_sampled type to be consistent with API documentation #830

Closed cindy-peng closed 9 months ago

cindy-peng commented 9 months ago

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

Fixes #802 🦕 This is to fix the expected type and value for trace_sampled to be consistent with the documentation. According to the LogEntry API documentation, trace_sampled is expected to be a bool type. This has also been correctly documented in the logging-winston homepage.