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

fix(types): update / fix types declaration file #41

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

geopic
Copy link
Contributor

@geopic geopic commented Nov 19, 2020

Resolves #14, #21. Let me know if there are any issues.


Thank you for your contribution to the vue-unicons repo.
Before submitting this PR, please make sure:

  • Your code builds clean without any errors or warnings
  • Your commits follow the сonvention

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Refactor

Importantly

To avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on PR.

@antonreshetov
Copy link
Owner

@geopic there are conflicts here, please fix them if you want the PR to be accepted.

@geopic
Copy link
Contributor Author

geopic commented Apr 1, 2021

I'll get down to it soon @antonreshetov 👌

@geopic
Copy link
Contributor Author

geopic commented Apr 2, 2021

Sorted @antonreshetov

@antonreshetov
Copy link
Owner

@geopic Please read the README, version 3 has changes that are not represented in your code.

@dwelch2344
Copy link

@geopic The src folders are deleted before bundling, so won't this fail for anyone who npm installs?

@martn001
Copy link

martn001 commented Jan 30, 2023

@geopic could this pr still be fixed and merged? And could it be used for Vue3?

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.

Typings?
4 participants