Skip to content

Conversation

@knoll3
Copy link
Contributor

@knoll3 knoll3 commented Dec 9, 2018

Hello! Interesting project. I was just looking through the readme and found some typos. No big deal, just wanted to help out and contribute something. There are also some grammatical errors but I just wanted to keep this PR limited to spelling errors. Thanks for taking the time to look at this.

@coveralls
Copy link

coveralls commented Dec 9, 2018

Pull Request Test Coverage Report for Build 189

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 95.4%

Totals Coverage Status
Change from base Build 188: 0.3%
Covered Lines: 640
Relevant Lines: 657

💛 - Coveralls

@Keyang Keyang merged commit 7c3f557 into Keyang:master Dec 9, 2018
@Keyang
Copy link
Owner

Keyang commented Dec 9, 2018

Thanks for the correction

@knoll3
Copy link
Contributor Author

knoll3 commented Dec 10, 2018

You're welcome!

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