Skip to content

Comments

fix: correct spelling mistakes found by CodeSpell#153

Open
abhi-14gyan wants to merge 1 commit intom-lab:mainfrom
abhi-14gyan:fix/codespell-typos
Open

fix: correct spelling mistakes found by CodeSpell#153
abhi-14gyan wants to merge 1 commit intom-lab:mainfrom
abhi-14gyan:fix/codespell-typos

Conversation

@abhi-14gyan
Copy link

Summary

Fix 6 spelling typos across library documentation, comments, and docstrings. All issues were identified by running CodeSpell on the repository.

No logic or behavior changes — only comments, docstrings, and documentation are affected.

Changes

File Line Before After
library/README.md 181 informations information
library/src/iqb/calculator.py 85 interation iteration
library/src/iqb/cli/cache_status.py 35 modifed modified
library/src/iqb/cli/cache_status.py 35 on dish on disk
library/src/iqb/pipeline/init.py 98 pipline pipeline
library/src/iqb/pipeline/pipeline.py 117 synching syncing

How it was found

codespell --skip=".git,*.pyc,__pycache__,.mypy_cache,.ruff_cache,*.egg-info,uv.lock,*.geojson,*.csv" .

Note: CodeSpell also flags THA in prototype/pages/IQB_Map.py:102. This is a false positive — it is Thailand's ISO 3166-1 alpha-3 country code used as a dictionary key.

Testing

  • ruff check . passes
  • uv run pytest — 276 passed, 11 pre-existing failures (unrelated to this change)
  • Re-running CodeSpell shows only the THA false positive

Fix 5 typos in comments, docstrings, and documentation:

- informations -> information (library/README.md)

- interation -> iteration (calculator.py)

- modifed -> modified, dish -> disk (cache_status.py)

- pipline -> pipeline (pipeline/__init__.py)

- synching -> syncing (pipeline.py)
@bassosimone
Copy link
Collaborator

Thank you, this diff looks good to me and I will eventually pick it up for merging.

@abhi-14gyan
Copy link
Author

@bassosimone Happy to help.

@bassosimone
Copy link
Collaborator

After listing our project for GSoC, we received a large amount of pull requests across several repositories. We are dealing with the backlog, but this would take time. We will get back to this pull request eventually. In the meanwhile, if you are a GSoC applicant, please read our updated GSoC policy: https://github.com/m-lab/gsoc/.

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.

2 participants