pnp/cli-microsoft365

Bug: superseding rules works incorrectly

Ouverte

#2 593 ouverte le 16 juil. 2021

 (6 commentaires) (0 réaction) (0 personne assignée)TypeScript (399 forks)auto 404
bughelp wantedkeep-open

Métriques du dépôt

Stars
 (1 390 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur