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
ci(docs-infra): run cli docs examples concurrently #29103
Conversation
bb80a41
to
676f721
Compare
@IgorMinar @gkalpak I brought down the non-ivy docs example tests job to 3min and the Ivy docs examples to 4min. This will be ultimately solved by Bazel anyway, but for now this is most likely even faster than with Bazel. so we have some good baseline and it helps with #28940 😁 |
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.
Left some minor comments. LGTM otherwise 👍
676f721
to
65207c0
Compare
@gkalpak Addressed your feedback. thanks! Note: This PR still needs approval from |
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 from i18n side 👍
@AndrewKushnir, what about fw-core? 😁 |
Yeah, still needs review from @angular/fw-core I think. Can someone please have a look? |
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
caretaker note: merging to master branch since patch didn't apply cleanly. @devversion is creating a new PR for patch |
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. |
No description provided.