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

pnpm support added & fixed a lot of bugs like build command #276

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

Conversation

BaderBC
Copy link

@BaderBC BaderBC commented Mar 5, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Feature:
    I added pnpm tool support

Bug:
I fix a lot of bugs like wrong types, build problems etc.

  • What is the current behavior? (You can also link to an open issue here)
    pnpm support as package manager #272 (no pnpm support)
    moreover "npm run build" don't works

  • What is the new behavior (if this is a feature change)?
    you can use pnpm to create your project

  • Other information:
    I updated some packages to newer version

@BaderBC
Copy link
Author

BaderBC commented Mar 6, 2023

@bitjson
I fixed all errors, fixed scripts in package.json, added pnpm support and finally fixed all test, but still have no idea why your CI fails.

Could you do code review (optionally fix CI) and merge?

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