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.

Sign up for GitHub

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

Jump to bottom

Removed minor spelling mistake in PR template #165

Merged
merged 2 commits into from
May 3, 2021
Merged

Removed minor spelling mistake in PR template #165

merged 2 commits into from
May 3, 2021

Conversation

raashika03
Copy link
Contributor

Checklist

  • I only changed lines related to my PR (no bulk reformating)
  • I indicated the source and check the license if I re-use code, or I did not re-use code
  • I made my best to solve only one issue in this PR, or explain why multi had to be solved at once.

@raashika03
Copy link
Contributor Author

Do not reformat code that you would not changing otherwise with you commit.
This line is not so clear mentioned in Contributing.md
Shouldn't it be like Do not reformat code that you would not be changing with your commit.
@bryan-brancotte

@bryan-brancotte bryan-brancotte merged commit d32c948 into edamontology:main May 3, 2021
bryan-brancotte added a commit that referenced this pull request May 3, 2021
@bryan-brancotte
No bulk reformatting
9816b49 
Thanks to @raashika03 for spoting this unclear sentence with #165 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@raashika03 @bryan-brancotte