Skip to content

Conversation

@rlundeen2
Copy link
Contributor

@rlundeen2 rlundeen2 commented Jan 6, 2026

  • Added print_deprecation_message (updated from an unused method)
  • Updated inconsistent log warnings to use this. It allows us to easily see warnings in tests, because previously we often did logger.warn
  • Went through and fixed all deprecated usage in tests and classes, other than those tests that were explicitly testing they existed

There are some flaky tests, but those are fixed in #1304; I want to merge that first

The journey was from something like 100 warnings originally -> 24 warnings -> 100 warnings when I explicitly called warn -> 30 warnings

Copy link
Contributor

@jsong468 jsong468 left a comment

Choose a reason for hiding this comment

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

Looks good!

@rlundeen2 rlundeen2 merged commit 0af160c into Azure:main Jan 8, 2026
20 checks passed
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.

4 participants