pnp/cli-microsoft365

Bug: superseding rules works incorrectly

Offen

#2.593 geöffnet am 16.07.2021

 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (399 Forks)auto 404
bughelp wantedkeep-open

Repository-Metriken

Stars
 (1.390 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

Discussed in https://github.com/pnp/cli-microsoft365/discussions/2590

When upgrading SPFx project from 1.11 to 1.12, CLI won't suggest removing rush-stack-compiler 3.3. While the rule is there, it's superseded by installing rush-stack-compiler 3.7. The problem is, that currently superseding doesn't take into account the difference between installing and uninstalling the package. We need to think about how to handle it as currently superseding is handled centrally outside the rule.

One way could be to replace the array of superseded rule IDs with a method that takes a rule as an argument. That way, the rule could examine other rule's properties and return a bool true or false depending if the rule should be superseded or not.

Contributor Guide