Skip to content

chore: enable codespell check#48

Merged
mathetake merged 4 commits intoenvoyproxy:mainfrom
zhaohuabing:codespell
Dec 18, 2024
Merged

chore: enable codespell check#48
mathetake merged 4 commits intoenvoyproxy:mainfrom
zhaohuabing:codespell

Conversation

@zhaohuabing
Copy link
Copy Markdown
Member

Fixes: #28

Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing marked this pull request as draft December 18, 2024 06:31
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing marked this pull request as ready for review December 18, 2024 07:00
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@@ -0,0 +1 @@
./manifests/charts/ai-gateway-helm/crds/*
Copy link
Copy Markdown
Member

@mathetake mathetake Dec 18, 2024

Choose a reason for hiding this comment

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

if this is needed, then the spell check seems not running properly on the code comments in api.go? Since the typo maxmimum in api.go is the reason why https://github.com/envoyproxy/ai-gateway/actions/runs/12387688325/job/34577531455 is failing, we should fix the go file rather than ignoring the generated file.

Copy link
Copy Markdown
Member Author

@zhaohuabing zhaohuabing Dec 18, 2024

Choose a reason for hiding this comment

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

This comes from the upstream Gateway HTTPRoute and has been fixed in kubernetes-sigs/gateway-api#3380.
However, we need this workaround until the Gateway API is upgraded to the next version.

Besides, the generated CRDs can be safely ingnored because the original spell errors will always be found in the API source code first.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

aha ok, now that makes sense... thank you for clarifying huabing!

@zhaohuabing zhaohuabing marked this pull request as draft December 18, 2024 07:58
Signed-off-by: Huabing Zhao <zhaohuabing@gmail.com>
@zhaohuabing zhaohuabing marked this pull request as ready for review December 18, 2024 08:05
@mathetake mathetake merged commit 6f3fe96 into envoyproxy:main Dec 18, 2024
@zhaohuabing zhaohuabing deleted the codespell branch December 19, 2024 07:41
mathetake added a commit to mathetake/ai-gateway that referenced this pull request Aug 26, 2025
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.

ci: add codespell for spell checks

2 participants