Skip to content
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.

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

Normalize vendors.yaml, and don't spell-check it #4238

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Apr 2, 2024

  • The Vendors data file doesn't contain any descriptions. I see little value in running spell-check over it, so I've added the data file name to the sSpell ignore paths list.
  • Normalizes the vendors.yaml entries -- dropping unnecessary quotes.

There is no change in generated site files (other than the site timestamp):

$ (cd public && git diff -bw --ignore-blank-lines) | grep ^diff
diff --git a/site/index.html b/site/index.html

@chalin chalin added the CI/infra CI & infrastructure label Apr 2, 2024
@chalin chalin requested a review from a team as a code owner April 2, 2024 15:34
@chalin chalin changed the title [CI] Normalize vendors.yaml, and don't spell-check it Normalize vendors.yaml, and don't spell-check it Apr 2, 2024
@cartermp cartermp merged commit 7952ea8 into open-telemetry:main Apr 3, 2024
14 checks passed
@chalin chalin deleted the chalin-im-ci-spell-check-cleanup-2024-04-02 branch April 3, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants