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

Eliminate deprecation warnings on resource skipped messages #2296

Merged
merged 1 commit into from
Nov 9, 2017

Conversation

adamleff
Copy link
Contributor

@adamleff adamleff commented Nov 7, 2017

PR #2216 introduced some new tests that use the old syntax that was deprecated in #2235. This gets them in line and eliminates the deprecation warnings.

PR #2216 introduced some new tests that use the old syntax that was
deprecated in #2235. This gets them in line and eliminates the
deprecation warnings.

Signed-off-by: Adam Leff <[email protected]>
@adamleff adamleff requested a review from a team as a code owner November 7, 2017 22:18
@jerryaldrichiii
Copy link
Contributor

Looks good to me. Great catch @adamleff!

Copy link
Contributor

@chris-rock chris-rock left a comment

Choose a reason for hiding this comment

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

Thank you @adamleff

@chris-rock chris-rock merged commit afd2344 into master Nov 9, 2017
@chris-rock chris-rock deleted the adamleff/exception-message-deprecations branch November 9, 2017 11:25
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