Skip to content
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

refactor(react-interactive-tab): export all types and hooks from barrel file #257

Conversation

soniaboller
Copy link
Contributor

@soniaboller soniaboller commented Oct 31, 2024

barrel file for InteractiveTab should export all types and hooks, in addition to the component

@soniaboller soniaboller requested review from dmytrokirpa and a team as code owners October 31, 2024 21:39
@dmytrokirpa dmytrokirpa added the @fluent-contrib/react-interactive-tab Main label for the @fluentui-contrib/react-interactive-tab package label Nov 1, 2024
InteractiveTabProps,
InteractiveTabSlots,
InteractiveTabState,
InteractiveTabInternalSlots,
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we should export InteractiveTabInternalSlots, as it's Internal and the core react-tabs package doesn't export TabInternalSlots as well. Other than that, looks good to me!

@dmytrokirpa
Copy link
Contributor

@soniaboller, thank you for your contribution. To ensure a release is triggered post-merger, kindly add a "change file" to the PR. This can be accomplished by executing yarn change at the monorepo root, then selecting the patch release type and providing a brief message.

@dmytrokirpa dmytrokirpa enabled auto-merge (squash) November 1, 2024 16:26
@soniaboller
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@soniaboller soniaboller closed this Nov 1, 2024
auto-merge was automatically disabled November 1, 2024 17:27

Pull request was closed

@dmytrokirpa dmytrokirpa merged commit a781bbc into microsoft:main Nov 1, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@fluent-contrib/react-interactive-tab Main label for the @fluentui-contrib/react-interactive-tab package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants