Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

deps: remove is-lambda dependency #299

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Conversation

okx-code
Copy link
Contributor

@okx-code okx-code commented Jun 11, 2024

This dependency stopped being used in commit 7c25367, as the responsibility for the HTTP agent was moved outside this package, but the dependency was not removed from package.json.

@npmcli/agent doesn't use is-lambda either, so this does actually remove a dependency.

This dependency stopped being used in commit 7c25367, as the responsibility for the HTTP agent was moved outside this package, but the dependency was not removed from package.json.
@okx-code okx-code requested a review from a team as a code owner June 11, 2024 15:21
@wraithgar wraithgar changed the title fix: remove is-lambda dependency deps: remove is-lambda dependency Jun 11, 2024
@wraithgar wraithgar merged commit d255fcb into npm:main Jun 11, 2024
25 checks passed
@okx-code okx-code deleted the remove-is-lambda branch June 11, 2024 15:42
@github-actions github-actions bot mentioned this pull request Jun 11, 2024
reggi pushed a commit that referenced this pull request Sep 26, 2024
🤖 I have created a release *beep* *boop*
---


##
[14.0.0](v13.0.1...v14.0.0)
(2024-09-26)
### ⚠️ BREAKING CHANGES
* `make-fetch-happen` now supports node `^18.17.0 || >=20.5.0`
### Bug Fixes
*
[`a1beba7`](a1beba7)
[#318](#318) align to npm
10 node engine range (@reggi)
### Dependencies
*
[`aa5d07a`](aa5d07a)
[#318](#318) `ssri@12.0.0`
*
[`0113d1f`](0113d1f)
[#318](#318)
`proc-log@5.0.0`
*
[`f0db136`](f0db136)
[#318](#318)
`minipass-fetch@4.0.0`
*
[`be3558d`](be3558d)
[#318](#318)
`@npmcli/agent@3.0.0`
*
[`d255fcb`](d255fcb)
[#299](#299) remove
is-lambda dependency (#299)
### Chores
*
[`30ee575`](30ee575)
[#318](#318) run
template-oss-apply (@reggi)
*
[`253f3b9`](253f3b9)
[#309](#309) bump
@npmcli/eslint-config from 4.0.5 to 5.0.0 (@dependabot[bot])
*
[`58dc7ef`](58dc7ef)
[#295](#295) bump
@npmcli/template-oss to 4.22.0 (@lukekarrys)
*
[`f51f1de`](f51f1de)
[#311](#311) postinstall
for dependabot template-oss PR (@hashtagchris)
*
[`381750e`](381750e)
[#311](#311) bump
@npmcli/template-oss from 4.23.1 to 4.23.3 (@dependabot[bot])

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants