Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some minor style issues #14873

Merged
merged 1 commit into from
Nov 20, 2024
Merged

Fix some minor style issues #14873

merged 1 commit into from
Nov 20, 2024

Conversation

no1wudi
Copy link
Contributor

@no1wudi no1wudi commented Nov 20, 2024

Summary

Fix typo and comment style in:

  • audio/audio_comp.c
  • drivers/drivers_initialize.c
  • drivers/pipes/fifo.c

Impact

Typo fix only

Testing

CI

Fix typo and comment style in:
* audio/audio_comp.c
* drivers/drivers_initialize.c
* drivers/pipes/fifo.c

Signed-off-by: Huang Qi <[email protected]>
@github-actions github-actions bot added Area: Drivers Drivers issues Area: Audio Size: S The size of the change in this PR is small labels Nov 20, 2024
@nuttxpr
Copy link

nuttxpr commented Nov 20, 2024

[Experimental Bot, please feedback here]

Yes. The PR appears to meet the requirements, although the testing section could be more robust by including specific build hosts and targets, along with actual before/after logs demonstrating the lack of functional change (since this is a typo fix, the logs should ideally be identical or very similar). Simply stating "CI" is insufficient detail for a proper review.

@jerpelea jerpelea merged commit fd1b525 into apache:master Nov 20, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Audio Area: Drivers Drivers issues Size: S The size of the change in this PR is small
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants