Skip to content

Conversation

mosabua
Copy link
Member

@mosabua mosabua commented Oct 17, 2025

@mosabua mosabua requested a review from a team as a code owner October 17, 2025 21:09
@netlify
Copy link

netlify bot commented Oct 17, 2025

Deploy Preview for ornate-narwhal-088216 ready!

Name Link
🔨 Latest commit 5dabc0a
🔍 Latest deploy log https://app.netlify.com/projects/ornate-narwhal-088216/deploys/68f2b199e684a60007021055
😎 Deploy Preview https://deploy-preview-2690--ornate-narwhal-088216.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Collaborator

@SharpRake SharpRake left a 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
Copy link
Collaborator

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

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

Copy link
Collaborator

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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from Chainguard Libraries.
from Chainguard Libraries:

pnpm list
```

Find the downloaded package in `node_modules/.pnpm/commander@4.1.1`. The
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
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

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.

2 participants