Skip to content

Conversation

@yarikoptic
Copy link
Contributor

More about codespell: https://github.com/codespell-project/codespell .

I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.

CI workflow has 'permissions' set only to 'read' so also should be safe.

Overall summary with counts of what was fixed (ran before fixes introduced)

      1 attribure ==> attribute
      1 betwenn ==> between
      1 correspondance ==> correspondence
      1 detailled ==> detailed
      1 fot ==> for, fit, dot, rot, cot, got, tot, fog
      1 heigth ==> height
      1 inthe ==> in the
      1 lenght ==> length
      1 minimzing ==> minimizing
      1 optionnaly ==> optionally
      1 overlayed ==> overlaid
      1 recommanded ==> recommended
      1 shorcuts ==> shortcuts
      1 wich ==> which
      2 cacheing ==> caching
      2 currenty ==> currently
      2 discared ==> discarded, discard
      2 reduceable ==> reducible
      2 responsability ==> responsibility
      2 typicall ==> typically, typical
      3 specifiction ==> specification
      4 explicitely ==> explicitly
     86 overriden ==> overridden

…(but ignoring overall fail due to ambigous ones)

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w || :",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
…nteractively

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w -i 3 -C 4",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@zm711
Copy link
Collaborator

zm711 commented Oct 19, 2025

I support this in general, but I would still defer this to @rossant since I don't have enough time to support this repo currently.

@rossant rossant merged commit 1ddcd01 into cortex-lab:master Oct 20, 2025
4 checks passed
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.

3 participants