Skip to content

v1.20.x CI/CODESPELL: Skip submodule path (#11078)#11133

Closed
roiedanino wants to merge 1 commit intoopenucx:v1.20.xfrom
roiedanino:v1.20.x-codespell-fix
Closed

v1.20.x CI/CODESPELL: Skip submodule path (#11078)#11133
roiedanino wants to merge 1 commit intoopenucx:v1.20.xfrom
roiedanino:v1.20.x-codespell-fix

Conversation

@roiedanino
Copy link
Copy Markdown
Contributor

(cherry picked from commit 688f8eb)

What?

Port to v1.20.x @tvegas1 fix for codespell issues coming from submodules

Why?

We have multiple v1.20.x PRs failing because of that codespell issue

@roiedanino roiedanino self-assigned this Jan 21, 2026
@roiedanino roiedanino changed the title CI/CODESPELL: Skip submodule path (#11078) v1.20.x CI/CODESPELL: Skip submodule path (#11078) Jan 21, 2026
@roiedanino roiedanino requested review from gleon99 and tvegas1 January 21, 2026 10:28
@tvegas1
Copy link
Copy Markdown
Contributor

tvegas1 commented Jan 21, 2026

CI failing: actually, unfortunately this cannot clean gpunetio leftovers from other CI run as in v1.20.x it is not in submodule list.

@roiedanino
Copy link
Copy Markdown
Contributor Author

CI failing: actually, unfortunately this cannot clean gpunetio leftovers from other CI run as in v1.20.x it is not in submodule list.

Yup, my bad, there's some other fix needed for that

@tvegas1
Copy link
Copy Markdown
Contributor

tvegas1 commented Jan 21, 2026

@Alexey-Rivkin, from azure Get Sources task we get: warning: unable to rmdir 'src/uct/ib/mlx5/gdaki/gpunetio': Directory not empty, any way to force cleanup?

@gleon99 gleon99 closed this Jan 21, 2026
@gleon99
Copy link
Copy Markdown
Contributor

gleon99 commented Jan 21, 2026

Closing this one, it will be handled by @dpressle instead.

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.

3 participants