-
-
Notifications
You must be signed in to change notification settings - Fork 918
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
icons-angular package #1091
base: main
Are you sure you want to change the base?
icons-angular package #1091
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
What can we do to kickstart this PR again? It would be great to have an official package for angular. Can I do something? |
✅ All icons are valid! |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
What prevents this PR from being merged? |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
@codecalm or @BG-Software-BG what can we do to get this merged? |
It's not completely done yet. It needs some improvements and a test project, which is started but not finished.. I haven't had time to finish this package. I'll try to do this in the near future. |
I will try to make some time this (or next week) to dive into your code and possible tests 👍🏼 Would it be an idea to use Angular 18 and use new features like:
I could try to migrate the current code as well |
It's already prepared as standalone component. We can use signals and inject(), but I think that's not the main priority. I want this package to be compatible with Angular 16+. |
Since v16 (LTS) ends one week before v19 (08-11-2024). I would suggest only supporting 17+ That way we can use signals etc without problems and we can create a more future proof package. |
Ok, that's a good point. You can change this package to use new features of Angular 17. |
I tried to add some changes to this PR. But had some trouble to get the angular-test running. (can't find the local library etc.. ) So for the time being, added a PR to the unofficial tabler-icons-angular to upgrade it to v18: |
No description provided.