Skip to content

Conversation

@dayo09
Copy link
Contributor

@dayo09 dayo09 commented Jan 9, 2023

This change includes a TreeView into TreeDataProvider. This enables ONE Tree to use 'reveal node' command provided by TreeView instance.

ONE-vscode-DCO-1.0-Signed-off-by: Dayoung Lee [email protected]


For #1465

@dayo09 dayo09 requested a review from a team January 9, 2023 08:06
@dayo09 dayo09 added the 2 approvals 2 approvals required to be merged label Jan 9, 2023
This change includes a TreeView into TreeDataProvider.
This enables ONE Tree to use 'reveal node' command provided by TreeView instance.

ONE-vscode-DCO-1.0-Signed-off-by: Dayoung Lee <[email protected]>
@dayo09 dayo09 force-pushed the 0109-include-treeview branch from 0c0dd43 to b3870d6 Compare January 9, 2023 08:07
@dayo09 dayo09 changed the title [OneExplorer] Add reveal command [OneExplorer] Add reveal function Jan 9, 2023
Copy link
Contributor

@llFreetimell llFreetimell left a comment

Choose a reason for hiding this comment

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

LGTM

@llFreetimell llFreetimell requested a review from a team January 10, 2023 00:14
Copy link
Contributor

@yunjayh yunjayh left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark merged commit 79f7f10 into Samsung:main Jan 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 approvals 2 approvals required to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants