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

[!Breaking Change]: Migrate package manager to pnpm #575

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

adeyinkaezra123
Copy link
Contributor

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    CI update

  • Why was this change needed? (You can also link to an open issue here)
    More speed, less memory. Other reasons explained here

@vercel
Copy link

vercel bot commented Oct 12, 2023

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

Name Status Preview Comments Updated (UTC)
docs-gpt ❌ Failed (Inspect) Oct 14, 2023 7:33pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 14, 2023 7:33pm

@vercel
Copy link

vercel bot commented Oct 12, 2023

@adeyinkaezra123 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@adeyinkaezra123
Copy link
Contributor Author

@dartpain Check this out!

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #575 (2ed15d2) into main (31fb180) will not change coverage.
Report is 33 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #575   +/-   ##
=======================================
  Coverage   17.57%   17.57%           
=======================================
  Files          59       59           
  Lines        2720     2720           
=======================================
  Hits          478      478           
  Misses       2242     2242           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@adeyinkaezra123
Copy link
Contributor Author

Any update concerning this @dartpain ?

@dartpain
Copy link
Contributor

Sorry my bad, i thought you where working on it.
I personally prefer pnpm, so lets do this.
Mark this PR as ready for review once you are happy with it please.
Thank you!

@dartpain
Copy link
Contributor

Please try to solve this error trace

src/conversation/ConversationBubble.tsx(10,18): error TS2307: Cannot find module 'copy-to-clipboard' or its corresponding type declarations.

Error: Command "pnpm run build" exited with 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants