open-telemetry / opamp-go

OpAMP protocol implementation in Go
Apache License 2.0
140 stars 69 forks source link

docs: fix ClientSyncedState docstring with a valid link #285

Closed tpaschalis closed 3 months ago

tpaschalis commented 3 months ago

This section was removed when the StatusReport fields were moved over to AgentToServer on https://github.com/open-telemetry/opamp-spec/commit/bc9c8ba8f3281bf924449bd54a13742c683f5639

This PR updates the docstring for a newly added field and links back to the status reporting section of the spec.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.96%. Comparing base (dc5742f) to head (e0e0044).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #285 +/- ## ======================================= Coverage 75.96% 75.96% ======================================= Files 25 25 Lines 1656 1656 ======================================= Hits 1258 1258 Misses 291 291 Partials 107 107 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tigrannajaryan commented 3 months ago

Thanks @tpaschalis