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

fixed camel case headers and emoji error in an issue template #65

Merged
merged 3 commits into from
May 26, 2020

Conversation

jpvajda
Copy link
Contributor

@jpvajda jpvajda commented May 21, 2020

I saw a few typos that I wanted to fix. Also @mmfred mentioned all headers should be sentence case, not camel so I fixed that too.

@jpvajda jpvajda requested a review from zstix May 21, 2020 20:02
@@ -15,7 +15,7 @@ assignees: ''
### Description
A clear and concise description of what the bug is.

### Step To Reproduce
### Step to reproduce
Copy link
Contributor

Choose a reason for hiding this comment

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

s/b Steps to reproduce

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Darn..I'll fix it!

@jpvajda jpvajda requested a review from timglaser May 22, 2020 03:12
@jpvajda
Copy link
Contributor Author

jpvajda commented May 22, 2020

adding @timglaser for a review as I think Zack is out

Copy link
Contributor

@timglaser timglaser left a comment

Choose a reason for hiding this comment

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

One small item. Other than that, looks good to me!

@@ -1,5 +1,5 @@
---
name: "Documentation Issue \U0001F589"
name: "Documentation Issue \U0001F4DC"
Copy link
Contributor

Choose a reason for hiding this comment

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

"Bug report" above is sentence cased like the markdown headers in this PR which makes me think this too should be sentence case. Thoughts?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah let me fix those issue template titles to be sentence case. good catch.

@jpvajda
Copy link
Contributor Author

jpvajda commented May 22, 2020

@timglaser I fixed that issue your reported.

timglaser
timglaser previously approved these changes May 22, 2020
Copy link
Contributor

@timglaser timglaser left a comment

Choose a reason for hiding this comment

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

👍

@timglaser timglaser merged commit 1a9751c into master May 26, 2020
@timglaser timglaser deleted the minor_typos branch May 26, 2020 18:59
@nr-opensource-bot
Copy link
Contributor

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants