Spellcheck in CI  #46

Activity

1. arminsabouri

   added

   good first issueGood for newcomersGood for newcomers

   help wantedExtra attention is neededExtra attention is needed

   nixPull requests that update nix codePull requests that update nix code

   github_actionsPull requests that update GitHub Actions codePull requests that update GitHub Actions code

   on Oct 6, 2026
2. arminsabouri

   closed this as completedin #47on Oct 7, 2026
3. arminsabouri

   reopened this on Oct 7, 2026
4. arminsabouri commented on Oct 7, 2026

   

   arminsabouri

   on Oct 7, 2026

   CollaboratorAuthor

   More actions

   Leaving this open. We still need a general spell checker and to fix: #47 (comment)
5. yuval-block commented on Oct 7, 2026

   

   yuval-block

   on Oct 7, 2026

   Collaborator

   More actions

   #47 doesn't close this but i think can be merged anyway so posting here, but i'm also fine with these changes being added to it

   IMO, and this applies to fungi monorepo and concurrent-psbt repo as well (cc @Mshehu5):

   • keep typos tool, it's useful for rust code (though maybe not as useful on this repo it doesn't hurt)
   • evaluate and add a flake check based on cspell - i think that'll cover python & rust docs & comments, nix comments and markdown files iiuc?
   • alternatively add cargo-spellcheck for rust docs checking (also not so useful for this repo specifically but doesn't hurt to be consistent)

   @arminsabouri is cspell what you meant in the call? codespell seems to be hosted service whereas cspell seems to be a reasonably popular typescript tool included in nixpkgs
6. yuval-block commented on Oct 7, 2026

   

   yuval-block

   on Oct 7, 2026

   Collaborator

   More actions

   ah there seem to be multiple things called codespell... seems like something better left to an LLM to look into, going to stop opening tabs now
7. bc1cindy commented on Oct 7, 2026

   

   bc1cindy

   on Oct 7, 2026

   Member

   More actions

   I'll also set the @oakagent to check and post typos in one comment
8. yuval-block

   added 2 commits that reference this issue on Oct 20, 2026

   Add cspell to flake checks

   ...

   This issue will close once commit 7009691 is merged into the 'main' branch.

   7009691

   Add cspell to flake checks

   ...

   This issue will close once commit 6b1a4bf is merged into the 'main' branch.

   6b1a4bf
9. yuval-block

   closed this as completedin #71on Oct 20, 2026