Skip to content

test(mdc-tabs): add performance tests for mdc-tabs #20496

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

Merged
merged 5 commits into from
Sep 11, 2020

Conversation

wagnermaciel
Copy link
Contributor

No description provided.

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 4, 2020
@wagnermaciel wagnermaciel marked this pull request as ready for review September 4, 2020 15:29
@wagnermaciel wagnermaciel requested a review from a team as a code owner September 4, 2020 15:29
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

cc. @crisbeto as he built MDC tabs IIRC.

@devversion devversion added merge safe target: patch This PR is targeted for the next patch release labels Sep 7, 2020
],
ng_srcs = [
":app.module.ts",
":fake-tab-data.ts",
Copy link
Member

@devversion devversion Sep 8, 2020

Choose a reason for hiding this comment

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

Just a FYI: The usual convention is that colon is only used for Bazel target. Plain input files are without the colon.

Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@wagnermaciel wagnermaciel added the action: merge The PR is ready for merge by the caretaker label Sep 8, 2020
@andrewseguin andrewseguin merged commit e13a79d into angular:master Sep 11, 2020
andrewseguin pushed a commit that referenced this pull request Sep 11, 2020
* test(mdc-tabs): add performance tests for mdc-tabs

* use template string to format fake tab data correctly

* fix indentation and remove unnecessary providers

* fix selectors for mdc tabs benchmark

* remove ':' from source files

(cherry picked from commit e13a79d)
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Oct 12, 2020
@wagnermaciel wagnermaciel deleted the mdc-tabs branch January 14, 2021 19:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants