Skip to content

refactor(shellcheck): enable for all files and fix remaining warnings #1512

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
merged 1 commit into from
Jun 25, 2025
Merged

refactor(shellcheck): enable for all files and fix remaining warnings #1512

merged 1 commit into from
Jun 25, 2025

Conversation

gforsyth
Copy link
Contributor

@gforsyth gforsyth requested a review from a team as a code owner June 25, 2025 19:46
@gforsyth gforsyth added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jun 25, 2025
@gforsyth gforsyth requested a review from raydouglass June 25, 2025 19:46
@gforsyth gforsyth mentioned this pull request Jun 25, 2025
Closed
32 tasks
jameslamb
jameslamb approved these changes Jun 25, 2025
View reviewed changes
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

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

Thanks! Appreciate the comment you added in dask_cuda_worker.sh.

@jameslamb jameslamb removed the request for review from raydouglass June 25, 2025 23:00
@jameslamb
Copy link
Member

/merge

@rapids-bot rapids-bot bot merged commit 4ce0536 into rapidsai:branch-25.08 Jun 25, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jameslamb jameslamb jameslamb approved these changes

Assignees
No one assigned
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gforsyth @jameslamb