Skip to content

RDKEMW-12934: Test PR to validate workflow#8

Open
balav08 wants to merge 6 commits into
developfrom
feature/RDKEMW-12934
Open

RDKEMW-12934: Test PR to validate workflow#8
balav08 wants to merge 6 commits into
developfrom
feature/RDKEMW-12934

Conversation

@balav08

@balav08 balav08 commented Mar 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@balav08 balav08 requested a review from a team as a code owner March 9, 2026 06:20
Copilot AI review requested due to automatic review settings March 9, 2026 06:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR appears to be a workflow-validation/test change, introducing no functional behavior changes to the AVInput plugin implementation.

Changes:

  • Adds an extra blank line before AVInputImplementation::GetVRRSupport().

Comment thread plugin/AVInputImplementation.cpp Outdated
@github-actions

github-actions Bot commented Mar 18, 2026

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

Copilot AI review requested due to automatic review settings April 14, 2026 09:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@RajaLucy

Copy link
Copy Markdown
Contributor

I have read the CLA Document and I hereby sign the CLA

Copilot AI review requested due to automatic review settings April 14, 2026 10:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread plugin/AVInputImplementation.cpp Outdated
return Core::ERROR_NONE;
}


Copilot AI Apr 14, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This adds an extra blank line between method definitions, resulting in two consecutive blank lines. Consider keeping spacing consistent (single blank line) to reduce whitespace-only diffs.

Suggested change

Copilot uses AI. Check for mistakes.
static bool isAudioBalanceSet = false;
static int planeType = 0;


Copilot AI Apr 14, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This introduces an extra blank line, creating a double-empty-line gap between the static declarations and using namespace std;. Consider removing the additional blank line to avoid whitespace-only churn/noisy diffs.

Suggested change

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 16, 2026 05:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings April 20, 2026 15:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

* limitations under the License.
**/


Copilot AI Apr 20, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This extra blank line creates a double empty line before the includes, adding diff noise without improving readability. Consider keeping a single blank line between the header comment and the first #include.

Suggested change

Copilot uses AI. Check for mistakes.
static bool isAudioBalanceSet = false;
static int planeType = 0;


Copilot AI Apr 20, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This added blank line results in two consecutive empty lines between the static declarations and using namespace std;. Consider removing the extra whitespace to keep formatting consistent and avoid unnecessary diffs.

Suggested change

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants