-
Notifications
You must be signed in to change notification settings - Fork 38
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
Enable pnpm in tools #308
Enable pnpm in tools #308
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
tools/pnpm/plugin.yaml
Outdated
url: https://github.com/pnpm/pnpm/releases/download/${version}/pnpm-${os}-${cpu} | ||
definitions: | ||
- name: pnpm | ||
known_good_version: v8.6.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.
Can you omit the v here and add it to the download url?
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.
Addressed
tools/pnpm/pnpm.test.ts
Outdated
|
||
toolTest({ | ||
toolName: "pnpm", | ||
toolVersion: "v8.6.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.
Also omit the v here?
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.
Addressed
tools/pnpm/plugin.yaml
Outdated
url: https://github.com/pnpm/pnpm/releases/download/${version}/pnpm-${os}-${cpu} | ||
definitions: | ||
- name: pnpm | ||
known_good_version: v8.6.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.
Generally we want to make the known_good_version
a semver - so make this 8.6.1 and then have the URL be v${version}
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.
Addressed
No description provided.