Skip to content

Create check.yml #46

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

Merged
merged 1 commit into from
Sep 21, 2024
Merged

Create check.yml #46

merged 1 commit into from
Sep 21, 2024

Conversation

dawgcodes
Copy link
Contributor

update

@Tomato6966
Copy link
Owner

What does that do / fix / improve?

@appujet
Copy link
Contributor

appujet commented Sep 16, 2024

What does that do / fix / improve?

It is make PR pkg for install

@Tomato6966
Copy link
Owner

What is "PR pkg"

@appujet
Copy link
Contributor

appujet commented Sep 17, 2024

What is "PR pkg"

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

@Tomato6966
Copy link
Owner

@appujet You wanted to add smt to this pr

@appujet
Copy link
Contributor

appujet commented Sep 21, 2024

@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
Copy link
Owner

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 Tomato6966 merged commit 0a95c0a into Tomato6966:main Sep 21, 2024
1 check failed
@Tomato6966
Copy link
Owner

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
Copy link
Owner

@dawgcodes

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.

3 participants