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

Deprecate the DebugWriter class #2680

Closed
jnm2 opened this issue Jan 24, 2018 · 3 comments
Closed

Deprecate the DebugWriter class #2680

jnm2 opened this issue Jan 24, 2018 · 3 comments
Milestone

Comments

@jnm2
Copy link
Contributor

jnm2 commented Jan 24, 2018

Followup from #1944

The class should probably be removed at some point as we don't use it.

I'd like to either do that or remove the Compact Framework comment.

/// <summary>
/// DebugWriter is a TextWriter that sends its
/// output to Debug. We don't use Trace because
/// writing to it is not supported in CF.
/// </summary>
public class DebugWriter : TextWriter

@CharliePoole
Copy link
Contributor

I haven't used it for a long time! I used to do it in my nunitlite self-executing tests once in a while. Removing it of course risks breaking someone who has found it and used it.

@stevenaw
Copy link
Member

Would deprecating this via the obsolete attribute be desired? If so, this could be a good first issue

@jnm2
Copy link
Contributor Author

jnm2 commented Mar 14, 2020

That sounds good to me. @nunit/framework-team?

@mikkelbu mikkelbu changed the title Remove the DebugWriter class Deprecate the DebugWriter class Dec 2, 2020
wbratz pushed a commit to wbratz/nunit that referenced this issue Dec 14, 2020
rprouse added a commit that referenced this issue Dec 28, 2020
…lass

Adds obsolete comment to debug writer class issue #2680
@rprouse rprouse closed this as completed Dec 28, 2020
@rprouse rprouse added this to the 3.13 milestone Dec 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants