PR file list keeps the old base after the base branch changes outside VS Code
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 68/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- github, typescript, vscode
- Domaine
- developer-experience, tooling
Piste de recherche
Start by tracing the Pull Requests view refresh flow through pr.refreshList, createOrUpdatePullRequestModel(), and getChangeModels() in dist/extension.js, as identified in the issue. Check how cached file changes are invalidated when the base branch changes, then reproduce with a PR whose new base changes the file diff. Done means Refresh shows the same file changes as GitHub’s “Files changed” tab without reloading the window.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
- Extension version: 0.170.0
- VSCode Version: 1.142.0
- OS: Windows 11
- Repository Clone Configuration (single repository/fork of an upstream repository): single repository
- GitHub Product (GitHub.com/GitHub Enterprise version x.x.x): GitHub.com
Steps to Reproduce:
- Have an open PR listed in the Pull Requests view.
- Change its base branch outside VS Code, for example with
gh pr edit <number> --base <branch>, to a branch where the PR's files differ. - Expand the PR in the Pull Requests view, then click Refresh and expand it again.
Result: the file list still compares against the old base. In my case, files that the new base already has as renamed show as deleted and added. Only Developer: Reload Window fixes it. GitHub's "Files changed" tab shows them as modified.
Expected: Refresh shows the files against the new base.
From reading dist/extension.js (not debugged): pr.refreshList calls forceClearCache(), and createOrUpdatePullRequestModel() keeps the existing model and calls update(), which sets the new base. Nothing clears _fileChanges, and getChangeModels() reuses it while it isn't empty.
- Langage dominant
- TypeScript
- Étoiles
- 2.6k
- Forks
- 804
- Merge moyen
- 1 j 23 h
- PR mergées (30 j)
- 44
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de microsoft/vscode-pull-request-github
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
microsoft/vscode-pull-request-github#9072 ·
Les mainteneurs répondent en général sous 1 jour
-
Issue 3 — DocumentationOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
microsoft/vscode-pull-request-github#8792 ·
Les mainteneurs répondent en général sous 1 jour
-
Separate link openers for issues and pull requestsPeut-être pris @hediet l’a pris il y a 1 jour. Ouvertefeature-request
microsoft/vscode-pull-request-github#9082 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Add a setting to fully disable background PR state polling (causes editor to jump/scroll on focus regain)Peut-être pris @alexr00 l’a pris il y a 2 jours. Ouverte
microsoft/vscode-pull-request-github#9077 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
microsoft/vscode-pull-request-github#9075 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de microsoft/vscode-pull-request-github
Issues similaires
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
cursor/community-plugins#465 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
vetro-protocol/vetro-monorepo#978 ·
Les mainteneurs répondent en général sous 1 jour
-
ai-generated bug category:security effort:s priority:p3
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
LanternOps/breeze#9697 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
maniator/verticopolis#1303 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 73/100
components-web-app/docs#297 ·