Include peerDependencies in forEach

[stevecrozz]

What's the problem this PR addresses?

Address missing dependencies when using foreach --topological with peerDependencies.

It is my understanding that a peer dependency is no less of a dependency than a regular one. Workspace seems to be supported in peer dependencies, so I believe we should support them in foreach.

Resolves #3594

...

How did you fix it?

This change should allow us to include peerDependencies when using foreach --topological.

...

Checklist

• I have read the Contributing Guide.

I have read this, but I'm going to pause here for feedback before I pour more energy into docs, tests, etc.

• I have set the packages that need to be released for my changes to be effective.

• I will check that all automated PR checks pass before the PR gets reviewed.

[arcanis]

That's a breaking change, since it'll break any project where two workspaces have peer dependencies on one another (and more generally it'll change the iteration order in hard-to-predict ways).

[stevecrozz]

That's a breaking change, since it'll break any project where two workspaces have peer dependencies on one another (and more generally it'll change the iteration order in hard-to-predict ways).

I don't understand the two workspaces scenario you pose. Isn't it the same as two workspaces that have regular dependencies on one another? But either way, I agree that it will be a breaking change for anyone who relied on the current behavior of not traversing peer dependencies.