Skip to content

Change deprecation level from Warning to Error#151

Merged
ZacSweers merged 2 commits intouber:masterfrom
VisheshVadhera:vishesh/autoDisposeWith-deprecation-error
Dec 31, 2017
Merged

Change deprecation level from Warning to Error#151
ZacSweers merged 2 commits intouber:masterfrom
VisheshVadhera:vishesh/autoDisposeWith-deprecation-error

Conversation

@VisheshVadhera
Copy link
Contributor

@VisheshVadhera VisheshVadhera commented Dec 29, 2017

Closes #143

Realised #143 would be a good first issue. So tackling it first.

@CLAassistant
Copy link

CLAassistant commented Dec 29, 2017

CLA assistant check
All committers have signed the CLA.

@ZacSweers
Copy link
Collaborator

Can you remove the comments saying this will become in error in 0.6.0?

@VisheshVadhera
Copy link
Contributor Author

Ah! Totally missed them. I'll push another commit.

Copy link
Collaborator

@ZacSweers ZacSweers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ZacSweers ZacSweers merged commit c180236 into uber:master Dec 31, 2017
@VisheshVadhera VisheshVadhera deleted the vishesh/autoDisposeWith-deprecation-error branch December 31, 2017 06:03
@ZacSweers ZacSweers mentioned this pull request Feb 4, 2018
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.

3 participants