The committers listed above are authorized under a signed CLA.

• ✅ login: ronnathaniel / name: Ron Nathaniel (258ce75)
• ✅ login: galbash / name: Gal Bashan (9627978, 3307258, 47bdb64)

When this is approved, we may want to do the same for the python-contrib. I'll open an issue and submit a fix once I gets approval for this one making sure that it is the right way to do it.

I've broken this down to two commits so it's easier to review: the first adds codespell to the CI, and the second is just all the typo fixes

Also, I have added spellcheck as another workflow, let me know if you think it makes more sense to place it under the lint testenv. I felt like it was a bit outside of lint scope, but both sounds totally reasonable to me so up to approvers :)

There was another PR which fixed the typos. It got merged. Please rebase your branch. I think we should also add this in actions workflow.

@srikanthccv branch updated (via a merge but if we are squashing I think it doesn't matter, since it now detects 4 changed files, let me know if I am wrong). Regarding the actions workflow, I think it will be automatically added since the test.yml workflow invokes the tox command won't it? If not is the test.yml the right workflow to add it in/?

It runs the tox but as you added a new environment for spellcheck it's not run automatically. You should update this line https://github.com/open-telemetry/opentelemetry-python/blob/main/.github/workflows/test.yml#L95

Done. Two final questions to make sure this is the thing we want:

1. Just making sure again that it makes sense that it is a separate test env and not included in lint
2. I named it spellcheck so the name is not tied to the tool we are using but the value it provides, let me know if you'd rather me call the test env codecov too for simplicity

looks like test_batch_span_processor_reset_timeout fails on the sdk windows build. From a quick look it looks like the after_calls variables is not being initialized outside the loop, so if the if is never true, we get a reference to a non-declared var. Does it make sense to just initialize it to an empty list or will that harm the logic that the test is trying to achieve?