-
Notifications
You must be signed in to change notification settings - Fork 110
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
Conversation
Templates for issues
*REMEMBER TO SANITIZE YOUR PERSONAL INFO* | ||
|
||
``` | ||
ApiHandle::InitializeLogging(...) |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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(...) |
There was a problem hiding this comment.
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 :)
Templates for issues