I have a different suggestion, although I'm not sure if that's the common practice:

Given we don't intend to make modifications on these images, I wonder if they should be kept out of the repo (so we don't need to keep these binaries in the history), and only referenced in the markdown files using some permanent links.

For example, all the images in this doc works fine without having to be part of the repo https://github.com/open-telemetry/opentelemetry-cpp/blob/79a9471f5c6462cf5d013331e91f25455f824cd2/examples/prometheus/README.md.

I think a broken link error is coming from this doc:
https://github.com/open-telemetry/opentelemetry-demo/blob/main/docs/demo_screenshots.md

thx @cartersocha, I've missed that one.

ifferent suggestion, although I'm not sure if that's the common practice:

Given we don't intend to make modifications on these images, I wonder if they should be kept out of the repo (so we don't need to keep these binaries in the history), and only referenced in the markdown files using some permanent links.

For example, all the images in this doc works fine without having to be part of the repo

Sounds good @reyang, let me try this.

Added links and removed files. Thx for the suggestion @reyang!