-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Fix data masking tests that depended on the old profiler utils #12113
Conversation
✅ Docs Preview ReadyNo new or changed pages found. |
|
commit: |
size-limit report 📦
|
I cannot figure out why the API extractor step continues to fail. I have no changes locally when I run that job, nor did the auto cleanup seem to fix it. |
@jerelmiller this might be why
|
@phryneas oh goodness. I've combed over those logs so many times and somehow missed this every time. Thanks so much! I'll get a separate PR to address this since this is currently failing on the |
After merging
main
into therelease-3.12
branch, many of the data masking tests began failing since they were still using the old profiler testing utilities. This PR updates the data masking tests to use the new render stream library instead.