Skip to content

Fix whitespace lint errors#324

Merged
jithunnair-amd merged 2 commits into
masterfrom
fix_lint_errors
Apr 20, 2026
Merged

Fix whitespace lint errors#324
jithunnair-amd merged 2 commits into
masterfrom
fix_lint_errors

Conversation

@amd-sriram
Copy link
Copy Markdown
Collaborator

@amd-sriram amd-sriram commented Mar 27, 2026

Motivation

Fix the code so that the lint checks pass.

Technical Details

Checked the following lints

hooks:
-   id: trailing-whitespace
-   id: check-ast
-   id: check-json
-   id: check-merge-conflict
-   id: check-toml
-   id: check-yaml
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks trailing-whitespace --all-files 
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks check-ast --all-files 
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks check-json --all-files 
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks check-merge-conflict --all-files 
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks check-toml --all-files 
pre-commit try-repo https://github.com/pre-commit/pre-commit-hooks check-yaml --all-files 

Only trailing-whitespace fails
Total 181 files changed with exactly 2209 insertions and 2209 deletions

Test Plan

Run linting workflow, branch used in PR #320

Test Result

https://github.com/ROCm/apex/actions/runs/23805177830

Submission Checklist

@amd-sriram amd-sriram self-assigned this Mar 27, 2026
@amd-sriram amd-sriram marked this pull request as ready for review March 31, 2026 21:19
@jithunnair-amd
Copy link
Copy Markdown
Collaborator

Moved non-whitespace changes to #326

@jithunnair-amd jithunnair-amd changed the title Fix lint errors Fix whitespace lint errors Apr 20, 2026
@jithunnair-amd jithunnair-amd merged commit 8384492 into master Apr 20, 2026
1 check passed
@jithunnair-amd jithunnair-amd deleted the fix_lint_errors branch April 20, 2026 15:25
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.

2 participants