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

perf!: call executePackageManagerRequest directly #2

Closed

Conversation

merceyz
Copy link
Owner

@merceyz merceyz commented Mar 15, 2024

Opening as a draft for now.

Ref nodejs#425 (comment)

Depends on nodejs#428

github-actions bot and others added 30 commits April 7, 2023 12:32
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
…odejs#244)


Co-authored-by: Kristoffer K. <merceyz@users.noreply.github.com>
BREAKING CHANGE: remove support for Node.js 14.x
* fix: don't override exit codes

* chore: update nock files
Node.js segfaults when using `npm@>=9.7.0` and `v8-compile-cache`
```console
$ docker run -it node:20.3.0-slim corepack npm@9.7.1 --version
[SIGSEGV]
```
Bumps [semver](https://github.com/npm/node-semver) from 7.5.1 to 7.5.2.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/main/CHANGELOG.md)
- [Commits](npm/node-semver@v7.5.1...v7.5.2)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
This is to update the indirect vm2 dependency and resolve some
dependabot alerts.
Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Refactoring of the CLI interface

* Updates the Nock snapshots

* Regenerates the Nock files on Node 16

* Update README.md

* Adds --cache-only to corepack install -g

* Fixes hash generation
* chore: update test to handle case when env var is set

- Fixes nodejs#303

* revert

* delete env vars in setupTests.js

* remove newline
aduh95 and others added 27 commits February 17, 2024 01:35
Force users to opt-in into the package managers they want to support.
As more and more package managers are added to Corepack, it's more and
more unlikely that our users will be interested in _all_ the package
managers Corepack supports, and it is unreasonable to expect Corepack
maintainers would be able to perform security audits.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Maël Nison <nison.mael@gmail.com>
* test: store nocks in a sqlite3 database

* chore: remove nock files

* refactor: reuse statement

* Update tests/recordRequests.js

Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>

* refactor: only create statements when needed

* refactor: move nocks db up one level

* fix: close db on exit

---------

Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
Co-authored-by: Steven <steven@ceriously.com>
* Removes the DESIGN document

* Removes the design document from the readme
Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
…s#413)

* feat: Pins the package manager as it's used for the first time

* Adds a warning when auto-pinning the package manager version

* Updates wording
@merceyz merceyz changed the base branch from merceyz/chore/remove-unused-error to main March 15, 2024 19:14
@merceyz merceyz closed this Mar 15, 2024
@merceyz
Copy link
Owner Author

merceyz commented Mar 15, 2024

Didn't notice it was targetting my fork.

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.