microsoft / microsoft/vscode-pull-request-github

Comments attached to wrong lines

Ouverte
#8,374 5 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

info-needed
Langage dominant
TypeScript
Étoiles
2.6k
Forks
795
Merge moyen
1 j 4 h
PR mergées (30 j)
46

Description

  • Extension version: 0.124.1
  • VSCode Version: 1.107.1
  • OS: Fedora 42
  • Repository Clone Configuration (single repository/fork of an upstream repository): fork of an upstream repo
  • GitHub Product (GitHub.com/GitHub Enterprise version x.x.x): GitHub.com

See here for the affected PR and the comment: https://github.com/scylladb/scylladb/pull/27386/files#r2643211612

When viewing with a browser, the comment is in the right place, right where there are 7 lines removed and the one below:
for shard in iterate_over_describe_stream(dynamodbstreams, arn, time.time() + 60):
is added instead.
In VS Code the comment is shown associated to the wrong lines - to the new lines starting from 1409, not the old lines starting from 1409.

Steps to Reproduce:

  1. Install the "GitHub Pull Requests" extension to VS Code.
  2. View the affected PR in VS Code.
  3. Search for a string from the comment to see it in VS Code

The right placement when viewed with Chromium:
Image

The wrong placement when viewed in VS Code:
Image

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Reproduisez le commentaire mal placé dans l’extension GitHub Pull Requests en ouvrant PR #27386 dans VS Code et en recherchant le texte du commentaire. Comparez le placement de l’extension avec la vue Chromium, et considérez l’issue comme terminée lorsque le commentaire est rattaché aux lignes supprimées plutôt qu’aux nouvelles lignes commençant à 1409.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
typescript
Domaine
devtools
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.