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.

Sign up for GitHub

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

Deprecate TerminalReporter._tw #2803

Closed
nicoddemus opened this issue Sep 27, 2017 · 2 comments
Closed

Deprecate TerminalReporter._tw #2803

nicoddemus opened this issue Sep 27, 2017 · 2 comments
Labels
type: feature-branch new feature or API change, should be merged into features branch

Comments

@nicoddemus
Copy link
Member

For some time there's an alias writer, which is public and part of the API because plugins may need to consume it.

We should deprecate it and remove it.
@nicoddemus nicoddemus added type: feature-branch new feature or API change, should be merged into features branch type: refactoring internal improvements to the code labels Sep 27, 2017
nicoddemus added a commit to nicoddemus/pytest that referenced this issue Sep 27, 2017
@nicoddemus
Deprecate TerminalReporter._tw
0e87fea 
Fix pytest-dev#2803
@nicoddemus nicoddemus mentioned this issue Sep 27, 2017
Merged
nicoddemus added a commit to nicoddemus/pytest that referenced this issue Sep 27, 2017
@nicoddemus
Deprecate TerminalReporter._tw
3b30c93 
Fix pytest-dev#2803
@nicoddemus nicoddemus added type: deprecation feature that will be removed in the future and removed type: refactoring internal improvements to the code labels Sep 27, 2017
@nicoddemus
Copy link
Member Author

Fixed in #2804

@nicoddemus
Copy link
Member Author

Because we don't want to expose TerminalWriter further because we plan to eventually replace it by something else, this was reverted in #2945.

@nicoddemus nicoddemus mentioned this issue Nov 23, 2017
Merged
@nicoddemus nicoddemus removed the type: deprecation feature that will be removed in the future label Jan 6, 2018
This was referenced Jul 22, 2018
Closed
Closed
@pytestbot pytestbot mentioned this issue Jul 30, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: feature-branch new feature or API change, should be merged into features branch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nicoddemus