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

Jump to bottom

PathExtention should manage placeholder like AdditionalTestExecutionParam #66

Closed
cyrilbesse opened this issue Jul 19, 2016 · 3 comments
Closed

PathExtention should manage placeholder like AdditionalTestExecutionParam #66

cyrilbesse opened this issue Jul 19, 2016 · 3 comments
Assignees
@csoltenborn
Labels
enhancement

Comments

@cyrilbesse
Copy link

Hi,

In *.gta.runsettings file (or via the option windows), I need to add relative paths by rapport to the executable location in the PATH execution.

I propose to manage this syntax, in *.gta.runsettings:

<GoogleTestAdapter>
    <PathExtension>$(ExecutableDir)\..</PathExtension>
</GoogleTestAdapter>

with $(ExecutableDir) - executable directory containing the tests

Regards
Cyril
csoltenborn added a commit that referenced this issue Jul 20, 2016
@csoltenborn
PathExtension option now supports ExecutableDir placeholder (#66)
0ef6927
@csoltenborn csoltenborn self-assigned this Jul 20, 2016
@csoltenborn
Copy link
Owner

Great suggestion, and rather easy to implement! Feel free to download the vsix produced by this build if you want to test the enhancement or if you need it urgently...

@cyrilbesse
Copy link
Author

Hi,
Thanks for your reactivity.
Test and approved

@csoltenborn
Copy link
Owner

Sweet - will not take too long until it's going to be released...

csoltenborn pushed a commit that referenced this issue Oct 21, 2017
@spebl
Merge pull request #66 from Microsoft/dev/spebl/tel
42d11e0 
Remove telemetry collection from TAfGT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@csoltenborn csoltenborn

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@csoltenborn @cyrilbesse