Skip to content

Fix typos discovered by codespell#6294

Merged
pared merged 7 commits intotreeverse:masterfrom
cclauss:codespell
Jul 7, 2021
Merged

Fix typos discovered by codespell#6294
pared merged 7 commits intotreeverse:masterfrom
cclauss:codespell

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Jul 6, 2021

This PR was created with the command codespell --ignore-words-list="ba,datas,fo,uptodate" -w

Thank you for the contribution - we'll try to review it as soon as possible. 🙏

@cclauss cclauss requested a review from a team as a code owner July 6, 2021 20:55
@cclauss cclauss requested a review from pared July 6, 2021 20:55
@skshetry
Copy link
Collaborator

skshetry commented Jul 7, 2021

codespell seems to offer a pre-commit hook as well. @cclauss, what do you think? How is it in terms of false-positives?

@iterative/dvc WDYT?

@cclauss
Copy link
Contributor Author

cclauss commented Jul 7, 2021

The pre-commit works well.

The syntax codespell --ignore-words-list="ba,datas,fo,uptodate" can deal with any false reports.

Copy link
Contributor

@pared pared left a comment

Choose a reason for hiding this comment

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

LGTM, seems like we could add it to pre-commit.

Co-authored-by: Saugat Pachhai <suagatchhetri@outlook.com>
Copy link
Collaborator

@skshetry skshetry left a comment

Choose a reason for hiding this comment

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

Thanks @cclauss for the fixes. 🙂

@pared pared merged commit 40f24e5 into treeverse:master Jul 7, 2021
@cclauss cclauss deleted the codespell branch July 7, 2021 15:50
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

Comments