chore: remove runsAfter

[benmccann]

continuing to incrementally tackle #85

runsAfter seems like it's a duplicate of dependsOn and we can just use one of the two

[pkg-pr-new]

Open in Stackblitz

pnpm add https://pkg.pr.new/sveltejs/cli/@svelte-cli/ast-tooling@86

pnpm add https://pkg.pr.new/sveltejs/cli/sv@86

pnpm add https://pkg.pr.new/sveltejs/cli/@svelte-cli/core@86

commit: ff049c3

[AdrianGonz97]

These are not the same.

runsAfter only guarantees the execution order. This does not mean that the specified adder is always executed. In other words, if adder A specifies adder B in runsAfter, then IF adder B is a selected adder, it will run before adder A.

dependsOn adds a dependency onto another adder. This means that the specified adder will always be executed. So if adder A specifies adder B in dependsOn, then every time adder A is selected, adder B will also be executed even when it hasn't been selected.

[AdrianGonz97]

I realize that we're not using runsAfter anywhere anymore. We used to need it for the eslint and prettier adder (don't remember which of the two specified it), but I guess it wasn't needed. I'm fine with removing runsAfter and adding it back if it's needed in the future.

[manuel3108]

Nothing to add @AdrianGonz97 mentioned everything