-
Notifications
You must be signed in to change notification settings - Fork 101
Add minimal example for pnpm #2690
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for ornate-narwhal-088216 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most of my suggestions are small things, some of which can probably be ignored.
However, I hit an error that stopped me from being able to complete the tech test, but this may be just a misstep on my part. If that's the case and it's obvious to you where I went wrong, let me know and I can approve it.
from Chainguard Libraries. | ||
|
||
```shell | ||
pnpm add commander@4.1.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
things started going haywire for me here:
mark@mark-chainguard:~/pnpm-example$ pnpm add commander@4.1.1
WARN GET https://libraries.cgr.dev/javascript/commander error (ERR_INVALID_CHAR). Will retry in 10 seconds. 2 retries left.
WARN GET https://libraries.cgr.dev/javascript/commander error (ERR_INVALID_CHAR). Will retry in 1 minute. 1 retries left.
ERR_PNPM_META_FETCH_FAIL GET https://libraries.cgr.dev/javascript/commander: Invalid character in header content ["authorization"]
This error happened while installing a direct dependency of /home/mark/pnpm-example
There's a good chance I just missed something but afaict I was following the given instructions
|
||
### Minimal example project | ||
|
||
Use the following steps to create a minimal example project for pnpm with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I swear the pnpm site is trying to annoy me specifically: https://pnpm.io/
Navigating there from google, I've seen it styled as pnpm
, PNPm
, and pnPm
. I don't know why/how it would ever show up differently
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
their docs go with pnpm
so you're all set here, but why is it like this??
pnpm init | ||
``` | ||
|
||
For simple testing, you can use direct access and environment variables as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For simple testing, you can use direct access and environment variables as | |
For testing purposes, you can use direct access and environment variables as |
I'd avoid "simple" when possible
|
||
For simple testing, you can use direct access and environment variables as | ||
detailed in the [access documentation](/chainguard/libraries/access/#env). Once | ||
the environment variables are set, the following steps configure the registry |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the environment variables are set, the following steps configure the registry | |
the environment variables are set, the following steps configure registry |
``` | ||
|
||
Add dependencies for your project into the `package.json` file to test retrieval | ||
from Chainguard Libraries. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from Chainguard Libraries. | |
from Chainguard Libraries: |
pnpm list | ||
``` | ||
|
||
Find the downloaded package in `node_modules/.pnpm/commander@4.1.1`. The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Find the downloaded package in `node_modules/.pnpm/commander@4.1.1`. The | |
Following this, you can find the downloaded package in `node_modules/.pnpm/commander@4.1.1`. The |
Find the downloaded package in `node_modules/.pnpm/commander@4.1.1`. The | ||
commands also result in the creation of the lock file `pnpm-lock.yaml`. | ||
|
||
Adjust the registry configuration to use your repository manager and any other |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Adjust the registry configuration to use your repository manager and any other | |
You can adjust the registry configuration to use your repository manager and any other |
Fix https://github.com/chainguard-dev/internal/issues/5434