Add codespell to pre-commit hooks and fix spelling of existing files

[davorrunje]

Why are these changes needed?

There is a large number of typos and spelling errors in the repository that could have been easily caught by a spelling checker. This PR Integrates the codespell tool as a pre-commit hook and scan changed files for spelling errors.

Existing spelling errors are also fixed.

Related issue number

Closes #1162

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (40dbf31) 30.59% compared to head (92ff46e) 49.08%.
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1161       +/-   ##
===========================================
+ Coverage   30.59%   49.08%   +18.49%     
===========================================
  Files          30       30               
  Lines        4256     4256               
  Branches      983     1034       +51     
===========================================
+ Hits         1302     2089      +787     
+ Misses       2861     1968      -893     
- Partials       93      199      +106     

Flag	Coverage Δ
unittests	48.98% <ø> (+18.44%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[rickyloynd-microsoft]

Adding codespell to pre-commit catches 139 spelling errors. Most of them are actual misspellings but a few are not, like Empress, which codespell wants to change to Impress. Would this force us to explicitly exclude certain files from codespell in pre-commit-config.yaml?

[davorrunje]

Adding codespell to pre-commit catches 139 spelling errors. Most of them are actual misspellings but a few are not, like Empress, which codespell wants to change to Impress. Would this force us to explicitly exclude certain files from codespell in pre-commit-config.yaml?

@rickyloynd-microsoft I fixed most spelling errors and then excluded the whole notebook directory from checking. There are some utf-8 encoded parts that confuse codespell. Not perfect, but better than nothing.

[rickyloynd-microsoft]

After exclusion of the notebooks, I see no problems with codespell's other changes. I defer to Chi on whether to add codespell to pre-commit.