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

Issue template typo grammar changes #859

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jsoberg
Copy link
Contributor

@jsoberg jsoberg commented Jun 23, 2021

I was looking through the code and noticed a small typo in the bug report template (Pastbin vs Pastebin). I figured I'd fix that and add some small grammatical changes that I noticed in the meantime.

@flocke flocke self-assigned this Jun 23, 2021
@flocke flocke added this to In progress in Administration via automation Jun 23, 2021
@ziegenberg ziegenberg self-requested a review July 10, 2021 15:37
Copy link
Contributor

@ziegenberg ziegenberg left a comment

Choose a reason for hiding this comment

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

Hi!

Thank you for your contribution! Looks good to me. I would only suggest cleaning up the commit history only to have your one actual commit without the other merge commits.

@jsoberg jsoberg force-pushed the IssueTemplateTypoGrammarChanges branch from e36d685 to 02b10fa Compare July 11, 2021 15:39
@jsoberg
Copy link
Contributor Author

jsoberg commented Jul 11, 2021

Sorry about that, I merged through the github website and didn't notice it created those merge commits. Fixed to just be the one relevant commit

Copy link
Contributor

@ziegenberg ziegenberg left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@ziegenberg
Copy link
Contributor

No worries! Things happen and that's what review processes are for. So everything's fine!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
Administration
In progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants