TEST DO NOT MERGE - #370
Conversation
|
📋 PR Format Reminder
Expected: |
There was a problem hiding this comment.
Pull request overview
Adds a new test.h header file (apparently as a placeholder/test change).
Changes:
- Introduces
test.hcontaining a single copyright line.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1 @@ | |||
| Copyright (c) mr compliance | |||
|
b'## Copyright scan failure |
| # Copyright (c) mr compliance | ||
| RDK Management LLC | ||
|
|
||
| LGPL | ||
|
|
||
| Apache 2.0 |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (2)
test.h:4
- This file is not valid C/C++ header text as-is: lines like
RDK Management LLC/LGPLare not in a comment, so includingtest.hwould cause compilation errors. At minimum, wrap the header text in a block comment (or convert this into a non-.h documentation file).
# Copyright (c) mr compliance
RDK Management LLC
LGPL
test.h:6
- The licensing/copyright header is ambiguous/inconsistent: it lists both "LGPL" and "Apache 2.0" without specifying which license actually applies (or using an SPDX identifier). Other headers in this repo use a single explicit license block and often include
SPDX-License-Identifier: ...(e.g.,source/test/service_dhcp/service_dhcp_mock.h:1-17). Please clarify the intended license(s) and update this header accordingly (including a proper copyright holder name and year).
LGPL
Apache 2.0
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
|
b'## Copyright scan failure |
No description provided.