-
-
Notifications
You must be signed in to change notification settings - Fork 724
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
GH2297 - Extended NUnit3Settings to support alternative app.config #2298
GH2297 - Extended NUnit3Settings to support alternative app.config #2298
Conversation
josiahpeters
commented
Sep 19, 2018
- Modified NUnit3Settings class to support new ConfigurationFile property that maps to --configfile=app.config for the nunit-console runner
- In the 3.8.0 release of the nunit-console runner, a new command line argument was added that supports loading the app.config file from an alternative path or file.
- Here is the new 3.8.0 feature: No way to specify app.config with console runner nunit/nunit-console#246
3817e2e
to
f2e6ae9
Compare
f2e6ae9
to
eab5df1
Compare
417b6c2
to
43e2793
Compare
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.
LGTM! 👍
/// Gets or sets a value indicating the path to an alternative app.config file to load. | ||
/// </summary> | ||
/// <value>The location that NUnit should load an alternative app.config file from.</value> | ||
public FilePath ConfigurationFile { get; set; } |
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.
I was a little too quick approving this PR. Could we make this argument match the NUnit argument name and rename it to ConfigFile
?
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.
Sure thing.
- Modified NUnit3Settings class to support new ConfigFile property that maps to --configfile=app.config for the nunit-console runner - In the 3.8.0 release of the nunit-console runner, a new command line argument was added that supports loading the app.config file from an alternative path or file. - Here is the new 3.8.0 feature: nunit/nunit-console#246
43e2793
to
c94fe5d
Compare
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.
LGTM 👍
@josiahpeters your changes have been merged, thanks for your contribution 👍 |
Relates to #2297 |