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: Adds repo shared package @lucide/shared #1904

Merged
merged 22 commits into from
Mar 6, 2024
Merged

Conversation

ericfennis
Copy link
Member

Closes #1603

What is the purpose of this pull request?

  • Refactor centralize shared code

Description

This refactor adds the ability for a shared package across packages. This continues on the work from @SamstyleGhost.
It needed some extra work to "bundle" the code from the shared package into each package.

This resulted in refactoring the build process from the lucide-solid package, which wasn't able to correctly include the shared package in the build. The idea was not to publish the shared package on NPM because that would need extra build processes and create a lot of extra work in maintaining versions.

The shared package will be used for the upcoming features we want to add like creating icons from external iconNodes.

@github-actions github-actions bot added 📦 dependencies Pull requests that update a dependency file ⚛️ react package Lucide React Package 💎 vue package Lucide Vue package ⚛️ preact package About the Preact Package 🪝 solid package About the Solid package labels Feb 21, 2024
@github-actions github-actions bot added ⚛️ react native package About the React Native package 🪨 static package About the static package labels Feb 21, 2024
@ericfennis ericfennis marked this pull request as draft February 21, 2024 22:08
@github-actions github-actions bot added the 🧳 lucide package About the lucide package label Feb 23, 2024
@github-actions github-actions bot added the 🎨 icon About new icons label Feb 28, 2024
@github-actions github-actions bot added the 🫧 metadata Improved metadata label Feb 28, 2024
Copy link

github-actions bot commented Feb 28, 2024

Added or changed icons

icons/package-open.svgicons/shield-off.svg

Preview cohesion icons/arrow-up-right-square.svgicons/arrow-down-right-square.svg
icons/package-open.svgicons/shield-off.svg
icons/arrow-down-0-1.svgicons/candy-cane.svg
Preview stroke widths icons/package-open.svgicons/shield-off.svg
icons/package-open.svgicons/shield-off.svg
icons/package-open.svgicons/shield-off.svg
DPI Preview (24px) icons/package-open.svgicons/shield-off.svg
Icon X-rays icons/package-open.svgicons/shield-off.svg

@ericfennis ericfennis merged commit ad1accb into main Mar 6, 2024
13 of 14 checks passed
@ericfennis ericfennis deleted the shared-package branch March 6, 2024 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📦 dependencies Pull requests that update a dependency file 🎨 icon About new icons 🧳 lucide package About the lucide package 🫧 metadata Improved metadata ⚛️ preact package About the Preact Package ⚛️ react native package About the React Native package ⚛️ react package Lucide React Package 🪝 solid package About the Solid package 🪨 static package About the static package 💎 vue package Lucide Vue package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants