DRIVERS-2790 Add codespell checker #1491
Conversation
blink1073
requested review from
jmikola,
comandeo,
vector-of-bool,
jess-sig,
baileympearson,
abr-egn and
dariakp
and removed request for
a team
blink1073
removed request for
a team,
comandeo,
vector-of-bool,
jess-sig,
baileympearson,
abr-egn and
dariakp
| rev: "v2.2.6" | ||
| hooks: | ||
| - id: codespell | ||
| args: ["-L", "fle,re-use,merchantibility,synching,crate,nin,infinit,te"] |
There was a problem hiding this comment.
"merchantability" is a misspelling of "merchantability", but I suppose most of this is required for LICENSE.md.
| @@ -54,7 +54,7 @@ | |||
| "key_vault_data": [], | |||
| "tests": [ | |||
| { | |||
| "description": "FLE2 encrypt fails with mising key", | |||
| "description": "FLE2 encrypt fails with missing key", | |||
There was a problem hiding this comment.
Please remember to make a note of any changed spec tests when entering downstream changes for DRIVERS-2780. It's possible that drivers depend on existing test names for custom skip logic, in which case some updates may be required (languages can always ignore syncing if they like).
There was a problem hiding this comment.
The underlying ticket is closed, and I don't see the word mising used in this org. Do you think it is worth opening a new ticket? Or perhaps ignoring mising in this PR and making a new ticket?
There was a problem hiding this comment.
If "mising" isn't used anywhere in the org I think we can just ignore this. Good idea running a search.
I also didn't consider the DRIVERS ticket would have already been triaged, and past the stage to denote driver changes.
Please complete the following before merging:
clusters, and serverless).