open-telemetry / opentelemetry-go-build-tools

Build tools for use by the Go API/SDK, the collector, and their associated contrib repositories
https://opentelemetry.io
Apache License 2.0
33 stars 37 forks source link

chloggen prints all output to stdout by default #611

Closed mowies closed 10 minutes ago

mowies commented 1 hour ago

While doing some debugging, I noticed that chloggen outputs to stdout by default. That has a few issues in the repos that use this tool, e.g. the opentelemetry-collector repos where the output is redirected to file which doesn't work as it should.

Example:

chloggen update --config ./path/to/config --dry 1>stdout.md 2>stderr.md

This will result in stdout.md being empty and stderr.md having the changelog dry run content.

mowies commented 1 hour ago

I will provide a small fix for this