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

feat: add types to CodeSnippet and CodeSnippetSkeleton #15324

Merged
merged 21 commits into from Apr 8, 2024

Conversation

Tresau
Copy link
Contributor

@Tresau Tresau commented Dec 5, 2023

Closes #13549

Add TypeScript types for CodeSnippet and CodeSnippetSkeleton
Depends on #15213

Changelog

New

  • Types for CodeSnippet and CodeSnippetSkeleton

Changed

  • .js -> .tsx files

Removed

  • n/a

Testing / Reviewing

Verify that unit tests run
Verify that components run in storybook

@Tresau Tresau requested a review from a team as a code owner December 5, 2023 14:39
@Tresau Tresau marked this pull request as draft December 5, 2023 14:39
@andreancardona andreancardona marked this pull request as ready for review April 3, 2024 18:59
Copy link

netlify bot commented Apr 4, 2024

Deploy Preview for v11-carbon-react ready!

Name Link
🔨 Latest commit 3e301e4
🔍 Latest deploy log https://app.netlify.com/sites/v11-carbon-react/deploys/6613ebb654e3fc00089e2fe7
😎 Deploy Preview https://deploy-preview-15324--v11-carbon-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@tw15egan tw15egan added this pull request to the merge queue Apr 8, 2024
Merged via the queue into carbon-design-system:main with commit c0a358a Apr 8, 2024
20 checks passed
preetibansalui pushed a commit to tay1orjones/carbon that referenced this pull request Apr 24, 2024
…-system#15324)

* feat: add types for CodeSnippet and CodeSnippetSkeleton

* feat: add types to Copy and CopyButton

* refactor: update index files to ts

* fix: fix lint

* revert: do not include changes for Copy and CopyButton

---------

Co-authored-by: TJ Egan <tw15egan@gmail.com>
Co-authored-by: Andrea N. Cardona <cardona.n.andrea@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

Add TypeScript types to CodeSnippet, CodeSnippetSkeleton
4 participants