Skip to content

vmh: (cosmetic) fix formatting#10641

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
lyakh:vmh
Mar 23, 2026
Merged

vmh: (cosmetic) fix formatting#10641
lgirdwood merged 1 commit intothesofproject:mainfrom
lyakh:vmh

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Mar 20, 2026

Merge needlessly split lines and remove superfluous braces.

Merge needlessly split lines and remove superfluous braces.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
Copilot AI review requested due to automatic review settings March 20, 2026 16:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Cosmetic refactor to simplify formatting in the VMH test by collapsing multi-line statements and removing braces around a single-statement if.

Changes:

  • Removed braces around a single-statement if (expect_success) branch
  • Collapsed a multi-line zassert_not_null() call into a single line

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@lgirdwood lgirdwood merged commit 1df6666 into thesofproject:main Mar 23, 2026
47 of 56 checks passed
@lyakh lyakh deleted the vmh branch March 23, 2026 15:58
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