Skip to content

chore: add basic spell checking as a pre-commit hook#2654

Open
MortGron wants to merge 1 commit into
masterfrom
codespell
Open

chore: add basic spell checking as a pre-commit hook#2654
MortGron wants to merge 1 commit into
masterfrom
codespell

Conversation

@MortGron
Copy link
Copy Markdown
Contributor

Description

Add basic spell checking as a pre-commit hook. Only enabled for the cognite/ folder. All flagged speclling mistakes in the same folder has been corrected.

Checklist:

  • Tests added/updated.
  • Documentation updated. Documentation is generated from docstrings - these must be updated according to your change.
    If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.
  • The PR title follows the Conventional Commit spec.

@MortGron MortGron marked this pull request as ready for review May 29, 2026 14:22
@MortGron MortGron requested review from a team as code owners May 29, 2026 14:22
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request integrates the codespell pre-commit hook to automatically detect and prevent spelling errors in the cognite/ directory. Additionally, it fixes various typos across multiple API and utility files, including corrections in docstrings and comments. Since there are no review comments, I have no feedback to provide.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.11%. Comparing base (d1ed829) to head (3cfee7c).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2654      +/-   ##
==========================================
- Coverage   93.12%   93.11%   -0.01%     
==========================================
  Files         490      490              
  Lines       49869    49869              
==========================================
- Hits        46439    46436       -3     
- Misses       3430     3433       +3     
Files with missing lines Coverage Δ
cognite/client/_api/annotations.py 95.65% <ø> (ø)
cognite/client/_api/data_modeling/containers.py 98.14% <ø> (ø)
cognite/client/_api/events.py 100.00% <ø> (ø)
cognite/client/_api/hosted_extractors/jobs.py 88.31% <ø> (ø)
cognite/client/_api/iam/groups.py 96.87% <ø> (ø)
cognite/client/_api/units.py 100.00% <ø> (ø)
cognite/client/_sync_api/annotations.py 100.00% <ø> (ø)
...gnite/client/_sync_api/data_modeling/containers.py 97.36% <ø> (ø)
cognite/client/_sync_api/events.py 100.00% <ø> (ø)
cognite/client/_sync_api/hosted_extractors/jobs.py 97.72% <ø> (ø)
... and 6 more

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@haakonvt haakonvt left a comment

Choose a reason for hiding this comment

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

🦄 nice nice

@haakonvt haakonvt self-assigned this May 29, 2026
@haakonvt haakonvt added risk-review-ongoing Risk review is in progress waiting-for-team Waiting for the submitter or reviewer of the PR to take an action labels May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk-review-ongoing Risk review is in progress waiting-for-team Waiting for the submitter or reviewer of the PR to take an action

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants