Skip to content

Conversation

@ffMathy
Copy link

@ffMathy ffMathy commented Nov 23, 2020

No description provided.

@PRMerger10
Copy link
Contributor

@ffMathy : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Mikejo5000
Copy link
Collaborator

@pavelhorak @AbhitejJohn to check fix

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Nov 23, 2020
@pavelhorak
Copy link

I believe is correct. @nohwnd can you confirm? Would it make sense adding TestCaseFilter attribute into RunConfiguration element example?

@ffMathy
Copy link
Author

ffMathy commented Nov 24, 2020

See this. That's when it was merged.

microsoft/vstest#2356

@ffMathy
Copy link
Author

ffMathy commented Nov 25, 2020

It looks like it was undocumented on purpose according to #2356. However, I think it's a great feature and workaround for the issue here: microsoft/vstest#2625

@nohwnd
Copy link
Contributor

nohwnd commented Nov 30, 2020

Yup, undocumented on purpose.

We agreed that this option should remain undocumented

Bottom of this comment. microsoft/vstest#2356 (comment)

@Mikejo5000
Copy link
Collaborator

@nohwnd Closing based on preceding comments. Let me know if I missed something. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants