Skip to content

fix typos #260

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 23, 2024
Merged

fix typos #260

merged 1 commit into from
Jan 23, 2024

Conversation

flatcap
Copy link
Contributor

@flatcap flatcap commented Jan 23, 2024

I noticed a typo in /etc/lcovrc, so I ran codespell on the entire codebase.

This PR fixes 104 typos.
Note: I'm a native English speaker :-)

Thanks

@@ -501,7 +501,7 @@ OPERATION
data (requires --add-tracefile)
--map-functions List tracefiles with non-zero coverage for
each function (requires --add-tracefile)
--intersect PATTERN Set interection with tracefiles matching
Copy link
Collaborator

Choose a reason for hiding this comment

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

this one should be 'intersection'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@henry2cox
Copy link
Collaborator

Thanks for finding all of these.
I think that we can conclude:

  • I can't spell. (Mrs. Besse - my second grade teacher - would be dismayed.)
  • I can't type either. (Can't remember my tenth grade typing teacher's name...she wouldn't be thrilled either.)
  • Kind of embarrassing to misspell 'embarrassing'.
    Sigh.

There is one new typo in the PR - I will merge after it is fixed (or will fix it myself, then merge).

Thanks for your contribution!
Henry

Signed-off-by: Richard Russon <[email protected]>
@flatcap
Copy link
Contributor Author

flatcap commented Jan 23, 2024

hehe, you're welcome

@henry2cox henry2cox merged commit d981242 into linux-test-project:master Jan 23, 2024
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.

2 participants