Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

fix: make @libp2p/interface a dependency #159

Merged
merged 1 commit into from
Dec 29, 2023

Conversation

achingbrain
Copy link
Member

This should be a dep not a dev dep.

Refs #158

This should be a dep not a dev dep.

Refs #158
achingbrain added a commit to ipfs/aegir that referenced this pull request Dec 29, 2023
Runs the dep-check command twice - once omitting dev deps and only
checking non-test code, then again but only testing test code.

Should prevent future bugs similar to ipfs/helia-ipns#159
@achingbrain achingbrain merged commit 546ecf0 into main Dec 29, 2023
18 checks passed
@achingbrain achingbrain deleted the fix/make-libp2p-interface-a-dep branch December 29, 2023 20:51
achingbrain added a commit to ipfs/aegir that referenced this pull request Jan 6, 2024
Runs the dep-check command twice - once omitting dev deps and only checking non-test code, then again but only testing test code.

[depcheck](https://www.npmjs.com/package/depcheck) is a bit wonky in that it doesn't support whitelisting files to check, only blacklisting so we pass a configurable list of ignore patterns that exclude known test files while checking prod deps and then a list of known prod files while checking dev deps.

Should prevent future bugs similar to ipfs/helia-ipns#159

BREAKING CHANGE: previously undetected missing or unused deps will cause the dep-check command to fail
github-actions bot pushed a commit that referenced this pull request Jan 8, 2024
## [@helia/ipns-v4.0.0](https://github.com/ipfs/helia-ipns/compare/@helia/ipns-v3.0.1...@helia/ipns-v4.0.0) (2024-01-08)

### ⚠ BREAKING CHANGES

* uses multiformats v13 and helia v3, renames `dht` routing to `libp2p`

### Features

* update helia to v3 and multiformats to v13 ([#167](#167)) ([a0381b9](a0381b9))

### Bug Fixes

* make @libp2p/interface a dependency ([#159](#159)) ([546ecf0](546ecf0)), closes [#158](#158)
Copy link

github-actions bot commented Jan 8, 2024

🎉 This PR is included in version @helia/ipns-v4.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant