search

Tomato6966 / lavalink-client

Easy, flexible and feature-rich lavalink@v4 Client. Both for Beginners and Proficients.
https://tomato6966.github.io/lavalink-client/
MIT License
61 stars 22 forks source link

Create check.yml #46

Closed dawgcodes closed 2 months ago

dawgcodes commented 2 months ago

update

Tomato6966 commented 2 months ago

What does that do / fix / improve?

appujet commented 2 months ago

What does that do / fix / improve?

It is make PR pkg for install

Tomato6966 commented 2 months ago

What is "PR pkg"

appujet commented 2 months ago

What is "PR pkg"

https://github.com/stackblitz-labs/pkg.pr.new

Tomato6966 commented 2 months ago

@appujet You wanted to add smt to this pr

appujet commented 2 months ago

@Tomato6966

This PR looks good, but it only seems to create a package of the PR. You can check it out here: https://github.com/stackblitz-labs/pkg.pr.new.

Tomato6966 commented 2 months ago

Is it required to do with pnpm? Can we not use smt like bun or normal npm for packages? Since it's a container there isn't a cache and thus you cannot profit from pnpm technically

I'm currently considering to approve this pr otherwise

Tomato6966 commented 2 months ago

Run actions/setup-node@v4 Found in cache @ /opt/hostedtoolcache/node/20.17.0/x64 Environment details /usr/local/bin/pnpm store path --silent /usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509 throw new UsageError(This project is configured to use ${result.spec.name} because ${result.target} has a "packageManager" field); ^

UsageError: This project is configured to use yarn because /home/runner/work/lavalink-client/lavalink-client/package.json has a "packageManager" field at Engine.findProjectSpec (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509:21) at async Engine.executePackageManagerRequest (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23539:24) at async Object.runMain (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:24232:5) { clipanion: { type: 'usage' } }

Node.js v20.17.0 Error: /usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509 throw new UsageError(This project is configured to use ${result.spec.name} because ${result.target} has a "packageManager" field); ^

UsageError: This project is configured to use yarn because /home/runner/work/lavalink-client/lavalink-client/package.json has a "packageManager" field at Engine.findProjectSpec (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23509:21) at async Engine.executePackageManagerRequest (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:23539:24) at async Object.runMain (/usr/local/lib/node_modules/corepack/dist/lib/corepack.cjs:24232:5) { clipanion: { type: 'usage' } }

Node.js v20.17.0

Tomato6966 commented 2 months ago

@dawgcodes