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

Is RunSettingsArguments being used? #2376

Closed
eiriktsarpalis opened this issue Aug 16, 2019 · 6 comments · May be fixed by #2771
Closed

Is RunSettingsArguments being used? #2376

eiriktsarpalis opened this issue Aug 16, 2019 · 6 comments · May be fixed by #2771

Comments

@eiriktsarpalis
Copy link
Member

Description

It seems that the DotNet.TestOptions.RunSettingsArguments property is dead code?

@matthid
Copy link
Member

matthid commented Aug 17, 2019

Yes looks like it. Should we implement or remove it? I guess you can just use the Common field for this?

@eiriktsarpalis
Copy link
Member Author

eiriktsarpalis commented Aug 17, 2019 via email

@eiriktsarpalis
Copy link
Member Author

eiriktsarpalis commented Aug 17, 2019 via email

@matthid
Copy link
Member

matthid commented Aug 17, 2019

to be honest I don't know what arguments this should be used for and how it relates to the possible msbuild, dotnet cli and dotnet runtime arguments. Or where these RunSettingsArguments logically belong to. So a PR with examples would be appreciated. Or just some examples on what this should be used for (maybe with links to existing docs)?

@eiriktsarpalis
Copy link
Member Author

eiriktsarpalis commented Aug 17, 2019 via email

@github-actions
Copy link

There has not been any activity in this issue for the last 3 months so it will be closed in 14 days if there is no activity.

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

Successfully merging a pull request may close this issue.

2 participants