Skip to content

fixed a bug where typos check is checking go.mod and go.sum#321

Merged
github-actions[bot] merged 1 commit intollm-d:mainfrom
nirrozenbaum:typos-check
Aug 29, 2025
Merged

fixed a bug where typos check is checking go.mod and go.sum#321
github-actions[bot] merged 1 commit intollm-d:mainfrom
nirrozenbaum:typos-check

Conversation

@nirrozenbaum
Copy link
Collaborator

This is blocking PR #320 to sync with IGW latest RC.
we should exclude go.sum and go.mod from typos check

Signed-off-by: Nir Rozenbaum <nirro@il.ibm.com>
@kfswain
Copy link
Collaborator

kfswain commented Aug 29, 2025

/lgtm
/approve

I think llm-d uses prow now so I think this works?

@github-actions github-actions bot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 29, 2025
@nirrozenbaum
Copy link
Collaborator Author

it works 😃

it’s not pure prow like in kubernetes, it’s prow-alike github action

@github-actions github-actions bot merged commit dc05adf into llm-d:main Aug 29, 2025
5 checks passed
@nirrozenbaum nirrozenbaum deleted the typos-check branch August 29, 2025 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm "Looks good to me", indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants