Skip to content
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

fix spelling errors #655

Merged
merged 8 commits into from
Sep 27, 2021
Merged

fix spelling errors #655

merged 8 commits into from
Sep 27, 2021

Conversation

bbonev
Copy link
Contributor

@bbonev bbonev commented Sep 22, 2021

I have seen the spelling in the phone response text, the others should be safe to fix

@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #655 (71a290c) into master (16a50c8) will not change coverage.
The diff coverage is 0.00%.

❗ Current head 71a290c differs from pull request most recent head bdc5686. Consider uploading reports for the commit bdc5686 to get more accurate results

@@          Coverage Diff           @@
##           master    #655   +/-   ##
======================================
  Coverage    0.06%   0.06%           
======================================
  Files          69      69           
  Lines       42701   42701           
======================================
  Hits           27      27           
  Misses      42674   42674           

Copy link
Member

@nijel nijel left a comment

Choose a reason for hiding this comment

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

The man pages are generated from docs/manual, can you please do the fixes there as well?

@bbonev
Copy link
Contributor Author

bbonev commented Sep 26, 2021

sure, fixed some more

@AppVeyorBot
Copy link

Build gammu 996 failed (commit 3406636a89 by @bbonev)

@AppVeyorBot
Copy link

Build gammu 997 completed (commit 71a290c0b5 by @bbonev)

@nijel nijel merged commit e51d9b7 into gammu:master Sep 27, 2021
@nijel
Copy link
Member

nijel commented Sep 27, 2021

Merged, thanks for your contribution!

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