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

[v1] chore: cleaning up typings and organization #99

Merged
merged 9 commits into from Jun 6, 2023
Merged

Conversation

stramel
Copy link
Collaborator

@stramel stramel commented May 23, 2023

Working to resolve outstanding typing issues and re-organizing a bit.

Documents slot usage - #41

@stramel stramel self-assigned this May 23, 2023
@changeset-bot
Copy link

changeset-bot bot commented May 23, 2023

⚠️ No Changeset found

Latest commit: 7635aef

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link

vercel bot commented May 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api-astroicon ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 6, 2023 2:11am
astroicon ❌ Failed (Inspect) Jun 6, 2023 2:11am

@@ -140,6 +140,10 @@ import { Icon } from 'astro-icon/components';
<Icon name="annotation" /> <!-- will be red -->
```

### Using with Frameworks

Astro Icon can be used with other frameworks utilizing the [`slot` element](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/slot). You can read more about how to use Slots in Astro here. [Passing Children to Framework Components](https://docs.astro.build/en/core-concepts/framework-components/#passing-children-to-framework-components)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stramel stramel changed the title chore: cleaning up typings and organization [v1] chore: cleaning up typings and organization May 25, 2023
@natemoo-re natemoo-re force-pushed the ms/document-v1 branch 2 times, most recently from f7e458e to f68f8c3 Compare June 6, 2023 01:45
Base automatically changed from ms/document-v1 to v1 June 6, 2023 01:49
Copy link
Owner

@natemoo-re natemoo-re left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌 Great stuff, thank you!

@natemoo-re natemoo-re merged commit 324a839 into v1 Jun 6, 2023
6 of 7 checks passed
@natemoo-re natemoo-re deleted the ms/clean-up branch June 6, 2023 02:15
natemoo-re pushed a commit that referenced this pull request Jun 6, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
natemoo-re pushed a commit that referenced this pull request Dec 21, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
natemoo-re pushed a commit that referenced this pull request Dec 22, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
natemoo-re pushed a commit that referenced this pull request Dec 22, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
natemoo-re pushed a commit that referenced this pull request Dec 22, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
natemoo-re pushed a commit that referenced this pull request Dec 22, 2023
* chore: convert core package to typescript

* feat: remove Iconify requirement

* docs: update core README

* chore: update website to use latest code

* fix: improve typings and organization

* fix: generating icon type definitions

* chore(lint): Prettier fix

* fix readme

---------

Co-authored-by: Michael Stramel <michael.stramel@koyfin.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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.

None yet

2 participants