Conversation
Co-authored-by: dfeen87 <158860247+dfeen87@users.noreply.github.com> Agent-Logs-Url: https://github.com/dfeen87/HLV-RAPS/sessions/e2e1522c-2ef7-4f0b-a073-c0d8d90824e6
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
This PR updates the telemetry module’s header comment license declarations to align with the repository’s root LICENSE file (which is a non-commercial license).
Changes:
- Replace
License: MIT ...references withLicense: Non-Commercial (see LICENSE)across telemetry headers. - Standardize the telemetry headers to consistently reference the root
LICENSEfile.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| include/raps/telemetry/telemetry_sink.hpp | Updates license header line to Non-Commercial wording. |
| include/raps/telemetry/telemetry_run_directory.hpp | Updates license header line and adds (see LICENSE) reference. |
| include/raps/telemetry/telemetry_ring_buffer.hpp | Updates license header line to Non-Commercial wording. |
| include/raps/telemetry/telemetry_metadata.hpp | Updates license header line to Non-Commercial wording. |
| include/raps/telemetry/telemetry_logger.hpp | Updates license header line to Non-Commercial wording. |
| include/raps/telemetry/telemetry_event.hpp | Updates license header line to Non-Commercial wording. |
| include/raps/telemetry/jsonl_sink.hpp | Updates license header line to Non-Commercial wording. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * Telemetry sink interface: keeps the core logger decoupled from export mechanisms. | ||
| * | ||
| * License: MIT (see LICENSE) | ||
| * License: Non-Commercial (see LICENSE) | ||
| */ |
There was a problem hiding this comment.
PR title suggests the LICENSE file was updated, but this PR only changes per-file header comments. Please either update the PR title/description to reflect that this is a header comment consistency change, or include the intended LICENSE file modifications in this PR.
Pull request created by AI Agent