Skip to content

Commit

Permalink
berry no longer include peer dependencies in all dependencies (#2745)
Browse files Browse the repository at this point in the history
  • Loading branch information
chris-olszewski committed Nov 17, 2022
1 parent 2e67849 commit 335de9c
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 4 deletions.
3 changes: 0 additions & 3 deletions cli/internal/lockfile/berry_lockfile.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,9 +119,6 @@ func (l *BerryLockfile) AllDependencies(key string) (map[string]string, bool) {
for name, version := range entry.Dependencies {
deps[name] = version
}
for name, version := range entry.PeerDependencies {
deps[name] = version
}

return deps, true
}
Expand Down
2 changes: 1 addition & 1 deletion cli/internal/lockfile/berry_lockfile_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ func Test_AllDependencies(t *testing.T) {
assert.Assert(t, pkg.Found, "expected to find react-dom")
deps, found := lockfile.AllDependencies(pkg.Key)
assert.Assert(t, found, "expected lockfile key for react-dom to be present")
assert.Equal(t, len(deps), 3, "expected to find all react-dom direct dependencies")
assert.Equal(t, len(deps), 2, "expected to find all react-dom direct dependencies")
for pkgName, version := range deps {
pkg, err := lockfile.ResolvePackage("some-pkg", pkgName, version)
assert.NilError(t, err, "error finding %s@%s", pkgName, version)
Expand Down

0 comments on commit 335de9c

Please sign in to comment.