Skip to content

[Fix] Correct misspellings across source and test files#535

Merged
sajmera-pensando merged 1 commit intoROCm:mainfrom
leo8a:fix/misspellings-cleanup
May 9, 2026
Merged

[Fix] Correct misspellings across source and test files#535
sajmera-pensando merged 1 commit intoROCm:mainfrom
leo8a:fix/misspellings-cleanup

Conversation

@leo8a
Copy link
Copy Markdown
Contributor

@leo8a leo8a commented Apr 24, 2026

Summary

  • Fix 10 misspellings found by the misspell linter across 5 files:
    • occuredoccurred (3 instances in tests/e2e/exporter_test.go)
    • requriesrequires (internal/controllers/device_config_reconciler.go)
    • disbaleddisabled (internal/controllers/device_config_reconciler.go)
    • apliedapplied (api/v1alpha1/deviceconfig_types.go)
    • enbaleenable (3 instances in internal/utils_test.go)
    • opertoroperator (tests/e2e/cluster_test.go)

Signed-off-by: Leonardo Ochoa-Aday lochoa@redhat.com

@leo8a
Copy link
Copy Markdown
Contributor Author

leo8a commented Apr 24, 2026

/cc @yansun1996

Copy link
Copy Markdown
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

This PR fixes a set of misspell linter-reported typos across the GPU Operator codebase (controllers, API types comments, and e2e/unit tests), improving readability and log/message clarity without changing runtime behavior.

Changes:

  • Corrected misspellings in e2e test log messages and strings.
  • Corrected misspellings in controller log messages and comments.
  • Corrected misspellings in API type comments and unit test case descriptions.

Reviewed changes

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

Show a summary per file
File Description
tests/e2e/exporter_test.go Fixes “occured” → “occurred” in error log messages during pod listing.
tests/e2e/cluster_test.go Fixes “opertor” → “operator” in an e2e deployment log message.
internal/utils_test.go Fixes “enbale” → “enable” in unit test case descriptions.
internal/controllers/device_config_reconciler.go Fixes “requries”/“disbaled” typos in comments and log messages.
api/v1alpha1/deviceconfig_types.go Fixes “aplied” → “applied” in a CRD/API field comment.

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

Signed-off-by: Leonardo Ochoa-Aday <lochoa@redhat.com>
@leo8a leo8a force-pushed the fix/misspellings-cleanup branch from 77c6e78 to bee5640 Compare May 6, 2026 17:07
@sajmera-pensando sajmera-pensando merged commit a5f8de3 into ROCm:main May 9, 2026
3 checks passed
@leo8a leo8a deleted the fix/misspellings-cleanup branch May 9, 2026 14:59
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