Skip to content
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

Require npm >=8.5.2 #2340

Merged
merged 1 commit into from
Mar 2, 2023
Merged

Require npm >=8.5.2 #2340

merged 1 commit into from
Mar 2, 2023

Conversation

kainino0x
Copy link
Collaborator

@kainino0x kainino0x commented Mar 2, 2023

Prior to this version, the @webgpu/types dependency-by-revision (#2332) won't work because the integrity hash is system-dependent. Starting with 8.5.2, integrity checks are skipped for git dependencies (though integrity hashes are still generated, despite the comment in the thread). See:
npm/cli#2846 (comment)

If your npm version isn't high enough, it should generate a warning (just warning, not hard error) every time you run npm install or npm ci.

Prior to this version, the @webgpu/types dependency-by-revision won't
work because the integrity hash is system-dependent. Starting with
8.5.2, integrity checks are skipped for git dependencies (though
integrity hashes are still generated, despite the comment in the
thread). See:
npm/cli#2846 (comment)
@kainino0x kainino0x merged commit b616629 into gpuweb:main Mar 2, 2023
@kainino0x kainino0x deleted the npm-version branch March 2, 2023 19:30
ErichDonGubler pushed a commit to mozilla/gpuweb-cts that referenced this pull request Aug 16, 2023
Prior to this version, the @webgpu/types dependency-by-revision won't
work because the integrity hash is system-dependent. Starting with
8.5.2, integrity checks are skipped for git dependencies (though
integrity hashes are still generated, despite the comment in the
thread). See:
npm/cli#2846 (comment)
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.

1 participant