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: added TS support and related changes #119

Merged

Conversation

imzedi
Copy link
Contributor

@imzedi imzedi commented Jan 31, 2025

TypeScript Migration & Enhanced Build process

Migrates the library to TypeScript and improves build system while maintaining backward compatibility.

Changes

  • Migrated to TypeScript with full type definitions
  • Replaced webpack with tsup for better build performance
  • Added support for React 16+ through React 18 (can work on 19 too)
  • Added dual module support (CJS & ESM)
  • Improved documentation and examples
  • Removed unnecessary build configurations

New Features

  • TypeScript type definitions and interfaces
  • Multiple import pattern support
  • React 18 example code
  • Better tree-shaking support

Testing

✅ React 16 compatibility
✅ React 17 & 18 support verified
✅ All import patterns tested
✅ Bundle size optimized

Documentation

  • Updated README with TypeScript information
  • Added React 18 usage examples
  • Improved import pattern documentation

Breaking Changes

  • This update maintains full backward compatibility.

References

Contributors

Special thanks ❤️ to @Tooni @Azuky @willibix

Copy link

linear bot commented Jan 31, 2025

@imzedi imzedi marked this pull request as ready for review January 31, 2025 14:18
@imzedi imzedi requested a review from rankarpan January 31, 2025 14:32
@imzedi
Copy link
Contributor Author

imzedi commented Jan 31, 2025

Merging this PR to release branch to publish rc

@imzedi imzedi merged commit 6ef815a into release-2.2.3 Jan 31, 2025
@rankarpan rankarpan deleted the feature/is-3174-ts-support-in-react-unicons-repo branch February 3, 2025 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant