-
Notifications
You must be signed in to change notification settings - Fork 6.8k
test(cdk/testing): add includeAsyncTests flag #25542
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
Conversation
This is needed by some downstream custom angular harnesses that use this set of tests but also do not rely on Angular framework stabilization.
How are these tests used by the downstream custom Angular harnesses? I don't think we ship these to NPM |
@devversion we're copybara-ing this into google3, so its used there |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ahh, I was on a personal device and didn't know this being for a Google use-case. SGTM |
* test(cdk/testing): add includeAsyncTests flag This is needed by some downstream custom angular harnesses that use this set of tests but also do not rely on Angular framework stabilization. * test(cdk/testing): rework `includeAsyncTests` into `skipAsyncTests` * test(cdk/testing): fix stale param doc (cherry picked from commit 7faafb4)
* test(cdk/testing): add includeAsyncTests flag This is needed by some downstream custom angular harnesses that use this set of tests but also do not rely on Angular framework stabilization. * test(cdk/testing): rework `includeAsyncTests` into `skipAsyncTests` * test(cdk/testing): fix stale param doc
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This is needed by some downstream custom angular harnesses that use this set of tests but also do not rely on Angular framework stabilization.