Only fire the PRs changed event when the timeline has actually changed (#7242) (#7247)

alexr00 · web-flow · commit 538725ce381b · 2025-07-07T08:47:20.000-07:00
Fixes #7235
diff --git a/src/github/githubRepository.ts b/src/github/githubRepository.ts
@@ -1509,8 +1509,11 @@ export class GitHubRepository extends Disposable {
 					const newEvents = timelineEvents.filter(event => (eventTime(event) ?? 0) > oldEventTime);
 					allEvents = [...issueModel.timelineEvents, ...newEvents];
 				}
+				const oldTimeline = issueModel.timelineEvents;
 				issueModel.timelineEvents = allEvents;
-				this._onDidChangePullRequests.fire();
+				if (!oldLastEvent || (allEvents.length !== oldTimeline.length)) {
+					this._onDidChangePullRequests.fire();
+				}
 			}
 			return timelineEvents;
 		} catch (e) {

Original file line number	Diff line number	Diff line change
`@@ -1509,8 +1509,11 @@ export class GitHubRepository extends Disposable {`
`1509`	`1509`	`const newEvents = timelineEvents.filter(event => (eventTime(event) ?? 0) > oldEventTime);`
`1510`	`1510`	`allEvents = [...issueModel.timelineEvents, ...newEvents];`
`1511`	`1511`	`}`
	`1512`	`+ const oldTimeline = issueModel.timelineEvents;`
`1512`	`1513`	`issueModel.timelineEvents = allEvents;`
`1513`		`- this._onDidChangePullRequests.fire();`
	`1514`	`+ if (!oldLastEvent \|\| (allEvents.length !== oldTimeline.length)) {`
	`1515`	`+ this._onDidChangePullRequests.fire();`
	`1516`	`+ }`
`1514`	`1517`	`}`
`1515`	`1518`	`return timelineEvents;`
`1516`	`1519`	`} catch (e) {`