Skip to content

Fix pre-commit for trailing whitespace and end of file #306

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.

Sign up for GitHub

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

Jump to bottom
Merged
rapids-bot merged 1 commit into from
Aug 15, 2025
Merged

Fix pre-commit for trailing whitespace and end of file #306

rapids-bot merged 1 commit into from
Aug 15, 2025

Conversation

@rgsl888prabhu
Copy link
Collaborator

@rgsl888prabhu rgsl888prabhu commented Aug 15, 2025 •
edited
Loading

Description

The changes only applied to select files rather than all, so removing that list and updating all the files.

Checklist

  • I am familiar with the Contributing Guidelines.
  • Testing
    • New or existing tests cover these changes
    • Added tests
    • Created an issue to follow-up
    • NA
  • Documentation
    • The documentation is up to date with these changes
    • Added new documentation
    • NA

@rgsl888prabhu rgsl888prabhu self-assigned this Aug 15, 2025
@rgsl888prabhu rgsl888prabhu requested review from a team as code owners August 15, 2025 14:46
@rgsl888prabhu rgsl888prabhu added the non-breaking Introduces a non-breaking change label Aug 15, 2025
@rgsl888prabhu rgsl888prabhu requested a review from a team as a code owner August 15, 2025 14:46
@rgsl888prabhu rgsl888prabhu added the improvement Improves an existing functionality label Aug 15, 2025
@rgsl888prabhu rgsl888prabhu mentioned this pull request Aug 15, 2025
Merged
8 tasks
kaatish
kaatish approved these changes Aug 15, 2025
View reviewed changes
Copy link
Contributor

@kaatish kaatish left a comment

Choose a reason for hiding this comment

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

Looks good to me

@rgsl888prabhu
Copy link
Collaborator Author

rgsl888prabhu commented Aug 15, 2025

/merge

@rapids-bot rapids-bot bot merged commit d0b8e96 into NVIDIA:branch-25.10 Aug 15, 2025
75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tmckayus tmckayus tmckayus approved these changes

@kaatish kaatish kaatish approved these changes

@KyleFromNVIDIA KyleFromNVIDIA Awaiting requested review from KyleFromNVIDIA KyleFromNVIDIA is a code owner automatically assigned from NVIDIA/cuopt-ci-codeowners

@rg20 rg20 Awaiting requested review from rg20 rg20 is a code owner automatically assigned from NVIDIA/cuopt-engine-codeowners

Assignees

@rgsl888prabhu rgsl888prabhu

Labels

improvement Improves an existing functionality non-breaking Introduces a non-breaking change

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rgsl888prabhu @tmckayus @kaatish