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

Update ISSUE_TEMPLATE.md #1293

Merged
merged 3 commits into from
Nov 16, 2017
Merged

Update ISSUE_TEMPLATE.md #1293

merged 3 commits into from
Nov 16, 2017

Conversation

pvlakshm
Copy link
Contributor

Added a note calling out that this template is only to file issues on vstest CLI.

Description

Please add a meaningful description for this change.
Ensure the PR has required unit tests.

Related issue

Kindly link any related issues. E.g. Fixes #xyz.

Added a note calling out that this template is only to file issues on vstest CLI.
@@ -1,10 +1,13 @@
## Description
> Describe the issue you've observed
> **IMPORTANT**: if the defect is reproduced only in a workflow from within the Visual Studio IDE then do not report the issue here - instead, please report it using Visual Studio's "Send Feedback" option that can be accessed from the Help menu.
Copy link
Contributor

Choose a reason for hiding this comment

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

> Describe the issue you've observed
> **IMPORTANT**: if the defect is reproduced only in a workflow from within the Visual Studio IDE then do not report the issue here - instead, please report it using Visual Studio's "Send Feedback" option that can be accessed from the Help menu.
>
> For a defect reproducable from the vstest commanline, describe the issue you've observed.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: spelling commandline


## Steps to reproduce
> What steps can reproduce the defect?
> Please share the setup, commandline for vstest.console, sample project, target
> framework etc.
>
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: required?

@smadala
Copy link
Contributor

smadala commented Nov 16, 2017

@dotnet-bot test Windows_NT / Release Build please.

@smadala
Copy link
Contributor

smadala commented Nov 16, 2017

@dotnet-bot please test Windows_NT / Release Build.

@pvlakshm pvlakshm merged commit bb8657b into master Nov 16, 2017
@pvlakshm pvlakshm deleted the pvlakshm-patch-1 branch November 16, 2017 13:49
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