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

Remove deprecated twisted.protocols.oscar #701

Merged
merged 3 commits into from May 19, 2017
Merged

Conversation

rodrigc
Copy link
Contributor

@rodrigc rodrigc commented Feb 1, 2017

@tomprince
Copy link
Contributor

Thanks.

This was deprecated in 16.2 which was released 2016-05-18. Or deprecation policy indicates that we shouldn't remove this until after 2017-05-18. I'm going to close this for the moment, but feel free to reopen this after that.

@tomprince tomprince closed this Feb 1, 2017
@rodrigc rodrigc reopened this Feb 3, 2017
@rodrigc
Copy link
Contributor Author

rodrigc commented Feb 3, 2017

There is no need to close this. @glyph approved a similar review which cannot be merged until 5/19/2017,
so it is OK to leave it open and just specify that it cannot be merged until that date.

@codecov-io
Copy link

codecov-io commented Feb 3, 2017

Codecov Report

Merging #701 into trunk will increase coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            trunk     #701      +/-   ##
==========================================
+ Coverage   89.63%   89.93%   +0.29%     
==========================================
  Files         843      841       -2     
  Lines      147797   146978     -819     
  Branches    13062    12563     -499     
==========================================
- Hits       132478   132183     -295     
+ Misses      12938    12412     -526     
- Partials     2381     2383       +2

@rodrigc rodrigc closed this May 19, 2017
@rodrigc rodrigc reopened this May 19, 2017
@rodrigc rodrigc merged commit cc18a95 into trunk May 19, 2017
@rodrigc rodrigc deleted the 9024-rodrigc-oscar-removal branch May 19, 2017 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants