WIP: feat: use pnpm #1789

Open
dpschen wants to merge 1 commits from dpschen/frontend:feature/feat-use-pnpm into main
Collaborator

Not sure about deployment.

Not sure about deployment.
Owner

I've never used pnpm but it looks quite promising!

If we switch to it, we should switch over completely - that also includes the dockerfile and the CI. We also need to update the developer docs.

I've never used pnpm but it looks quite promising! If we switch to it, we should switch over completely - that also includes the dockerfile and the CI. We also need to update the [developer docs](https://vikunja.io/docs/development/#frontend-requirements).
Owner

I think the build image we use in CI also needs an upgrade.

I think the build image we use in CI also needs an upgrade.
dpschen force-pushed feature/feat-use-pnpm from b008e106d1 to 0bcfbdbacb 4 months ago
dpschen force-pushed feature/feat-use-pnpm from 0bcfbdbacb to 922badd4da 4 months ago
Owner

Seems fine to me. The only thing missing now is the pipeline image and likely the docker image itself too.

We'll either need to use an image from someone else or build one ourselves. @dpschen Do you know if there are any official docker images from pnpm?

Seems fine to me. The only thing missing now is the pipeline image and likely the docker image itself too. We'll either need to use an image from someone else or build one ourselves. @dpschen Do you know if there are any official docker images from pnpm?
Poster
Collaborator
I did find this: https://pnpm.io/cli/fetch#usage-scenario
Owner

That looks like we'll need to add a RUN curl -f https://get.pnpm.io/v6.16.js | node - add --global pnpm to our Dockerfile and also build a simple docker image with that so we can use pnpm in the pipeline without having to install it on every pipeline step.

That looks like we'll need to add a `RUN curl -f https://get.pnpm.io/v6.16.js | node - add --global pnpm` to our `Dockerfile` and also build a simple docker image with that so we can use pnpm in the pipeline without having to install it on every pipeline step.
dpschen added the
help wanted
label 3 months ago
dpschen force-pushed feature/feat-use-pnpm from 922badd4da to 361e0e22d5 1 month ago
Owner

We'll need to upgrade renovate as well because only the newer versions support pnpm.

We'll need to upgrade renovate as well because only the newer versions support pnpm.
Poster
Collaborator

We'll need to upgrade renovate as well because only the newer versions support pnpm.

Where can this be upgraded?

> We'll need to upgrade renovate as well because only the newer versions support pnpm. Where can this be upgraded?
Owner

I'll need to do that on the server.

I'll need to do that on the server.
Owner

Just upgraded, renovate should now be able to handle pnpm dependencies.

Just upgraded, renovate should now be able to handle pnpm dependencies.
Some checks failed
continuous-integration/drone/pr Build is failing
Required
Details
This pull request has changes conflicting with the target branch.
yarn.lock
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/frontend#1789
Loading…
There is no content yet.