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. #1100

Closed

Conversation

sebastic
Copy link
Contributor

@sebastic sebastic commented Jul 4, 2019

The lintian QA tool reported a spelling error for the Debian package build of GMT 6.0.0rc1:

  • reencode -> re-encode

 * reencode -> re-encode
@welcome
Copy link

welcome bot commented Jul 4, 2019

💖 Thanks for opening this pull request! 💖

Please make sure you read our Contributing Guide and abide by our Code of Conduct.

A few things to keep in mind:

  • If you need help writing tests, take a look at the existing ones for inspiration. If you don't know where to start, let us know and we'll walk you through it.
  • All new features should be documented. It helps to write the comments below for your functions describing the it, all arguments, and return types before writing the code. This will help you think about your code design and results in better code.
  • We are grateful that you put in the effort to do this! 🎉

PaulWessel added a commit that referenced this pull request Jul 4, 2019
This is based on PR #1100 but here also done in one of the source files.
@PaulWessel PaulWessel mentioned this pull request Jul 4, 2019
PaulWessel added a commit that referenced this pull request Jul 4, 2019
This is based on PR #1100 but here also done in one of the source files.
@PaulWessel
Copy link
Member

Thanks, I fixed these but via the original files.

@PaulWessel PaulWessel closed this Jul 4, 2019
@sebastic sebastic deleted the spelling-errors branch July 4, 2019 06:49
obaney pushed a commit to obaney/gmt that referenced this pull request Aug 18, 2021
Co-authored-by: Dongdong Tian <seisman.info@gmail.com>
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.

None yet

2 participants