dotnet/roslyn

CodeLens does not invalidate for changes in downstream projects

Ouverte

#49 775 ouverte le 3 déc. 2020

Voir sur GitHubGuide contributeur
 (0 commentaire) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEBugCodeLenshelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

Version Used: e9fd4dc7

Steps to Reproduce:

  1. Create project A:

    public static class First {
  public static void Method() { }
}

  2. Create project B, which references project A:

    public class Second {
  public void Method() {
    First.Method();
  }
}

  3. Observe the CodeLens reference count on First.Method

  4. Comment out Second.Method

  5. Observe the CodeLens reference count on First.Method

Expected Behavior:

  • At step (3), the reference count is 1
  • At step (5), the reference count is 0

Actual Behavior:

  • ✔ At step (3), the reference count is 1
  • ❌ At step (5), the reference count is 1

Additional Details:

The change in step (4) does not update the project version which is used here for data point invalidation:

https://github.com/dotnet/roslyn/blob/6eecbdbeff1a547709b53f5bdc93b60749c6b1ef/src/VisualStudio/Core/Def/Implementation/CodeLens/CodeLensCallbackListener.cs#L70

Guide contributeur

Direction de recherche
Examinez la logique d'invalidation dans CodeLensCallbackListener.cs, en particulier comment les changements de version du projet sont détectés pour les projets en aval.
Stack technique
csharp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70