Skip to content
This repository has been archived by the owner on Aug 3, 2022. It is now read-only.

Correct grammar to suit third person present tense #1127

Merged
merged 1 commit into from
Jun 12, 2017
Merged

Conversation

jae-huh
Copy link
Contributor

@jae-huh jae-huh commented Jun 11, 2017

No description provided.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 03909e4 on Jae-Huh:master into ** on okfn:master**.

@jae-huh
Copy link
Contributor Author

jae-huh commented Jun 11, 2017

Hmm I corrected line 6 to this. It's a minor change but the issue also shows what's changed. Could you explain why the change is unknown? (This is my first time contributing to an open source project so please bear with me :))

@brew
Copy link
Collaborator

brew commented Jun 12, 2017

Hi Jae-Huh. Coveralls is an automated system for reporting how much of the project code is covered by our tests, and reports how that coverage might change for a given Pull Request. The README isn't part of coverall's remit, so PRs that only make changes to the README confuse coveralls a bit. It's fine though.

Separately, can I suggest next time you add a description of your changes to the Pull Request message.

Thanks for the PR!

@brew brew merged commit 09d33c6 into okfn:master Jun 12, 2017
@jae-huh
Copy link
Contributor Author

jae-huh commented Jun 12, 2017

Thanks for letting me know, Brook. I'll make sure to add more details next time!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants