-
Notifications
You must be signed in to change notification settings - Fork 44
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
Dropdown::ListItem::Interactive
enhancements MAIN
#2347
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
|
packages/codemods/transforms/v4/dropdown-list-item-interactive/README.md
Outdated
Show resolved
Hide resolved
Dropdown::ListItem::Interactive
enhancementsDropdown::ListItem::Interactive
enhancements MAIN
b496f32
to
8725e93
Compare
14acd39
to
8725e93
Compare
961233b
to
42732fb
Compare
ff5f688
to
664c515
Compare
664c515
to
42f6b72
Compare
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.
I haven't reviewed everything, just made sure that the tests of the codemod were covering enough use cases. From what I see now the coverage is good (and I can't think if something extra not covered) so I am approving that part (and defer to the other reviewers to cover the whole PR). Good job (and particularly appreciated that you gave meaningful names to the test files π)
Co-authored-by: Heather Larsen <hlarsen@hashicorp.com> Co-authored-by: Jory Tindall <jory.tindall@hashicorp.com>
d6448ab
to
0c8e4b1
Compare
π nice one @LilithJames-HDS @zamoore |
π Summary
If merged, this PR will enhance the
Dropdown::ListItem::Interactive
by adding a yielding block.π οΈ Detailed description
This PR contains the following previously approved child PRs
π External links
Jira ticket: HDS-3620
π Component checklist
π¬ Please consider using conventional comments when reviewing this PR.