Skip to content

Update issue templates #96

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

Merged
merged 4 commits into from
Mar 9, 2020
Merged

Update issue templates #96

merged 4 commits into from
Mar 9, 2020

Conversation

KaibaLopez
Copy link
Contributor

Templates for issues

*REMEMBER TO SANITIZE YOUR PERSONAL INFO*

```
ApiHandle::InitializeLogging(...)
Copy link
Contributor

Choose a reason for hiding this comment

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

In their code it will show up as something like

apiHandle.InitializeLogging(...);

I don't know if there's a better way to communicate that the call isn't static but we don't know how they've created their ApiHandle instance.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yea but i'm thinking this is more of a hint on how to do it rather than instructions, that should be on the readme or documentation.

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's go with this, and see if it confuses people. If it does, we'll write it another way until it doesn't :)

A clear and concise description of what the bug is.

**Platform/OS you're using**
MacOs/Linux/Windows
Copy link
Contributor

Choose a reason for hiding this comment

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

Might be worth explicitly calling out device/hardware properties as well.

*REMEMBER TO SANITIZE YOUR PERSONAL INFO*

```
ApiHandle::InitializeLogging(...)
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's go with this, and see if it confuses people. If it does, we'll write it another way until it doesn't :)

@justinboswell justinboswell merged commit 327605b into master Mar 9, 2020
@justinboswell justinboswell deleted the KaibaLopez-issueTemplates branch March 9, 2020 21:21
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