Skip to content

Conversation

devversion
Copy link
Member

Follow-up that addresses feedback from the original tab-group
harness PR. See: #16728 (comment)

@devversion devversion added pr: merge safe target: patch This PR is targeted for the next patch release labels Aug 19, 2019
@devversion devversion requested a review from crisbeto as a code owner August 19, 2019 08:17
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Aug 19, 2019
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn jelbourn added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Aug 19, 2019
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

@@ -6,4 +6,6 @@
* found in the LICENSE file at https://angular.io/license
*/

export type TabGroupHarnessFilters = {};
export type TabGroupHarnessFilters = {
Copy link
Member

Choose a reason for hiding this comment

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

Should this have a doc string?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure. The boilerplate and other harnesses don't have a doc string for these either.

@devversion devversion force-pushed the feat/harness-tabs-filter-by-selected-tab branch from 9f4395c to 8d337ef Compare August 19, 2019 21:29
@devversion devversion removed the action: merge The PR is ready for merge by the caretaker label Aug 19, 2019
…lected tab

Follow-up that addresses feedback from the original tab-group
harness PR. See: angular#16728#discussion_r314819393
@devversion devversion force-pushed the feat/harness-tabs-filter-by-selected-tab branch from 8d337ef to e726ffe Compare August 19, 2019 21:39
@devversion devversion added the action: merge The PR is ready for merge by the caretaker label Aug 19, 2019
@andrewseguin andrewseguin merged commit dc2c700 into angular:master Aug 21, 2019
andrewseguin pushed a commit that referenced this pull request Aug 26, 2019
…lected tab (#16813)

Follow-up that addresses feedback from the original tab-group
harness PR. See: #16728#discussion_r314819393

(cherry picked from commit dc2c700)
@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 Sep 21, 2019
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