chore: format pr summary changes
This commit is contained in:
parent
689d4124c2
commit
e28aa0edf2
|
|
@ -113,8 +113,8 @@ export function PRSummaryParts({
|
|||
</div>
|
||||
) : null}
|
||||
</div>
|
||||
);
|
||||
})}
|
||||
);
|
||||
})}
|
||||
</div>
|
||||
);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -22,7 +22,8 @@ export function sessionScmSummaryQueryOptions(sessionId: string) {
|
|||
return {
|
||||
queryKey: sessionScmSummaryQueryKey(sessionId),
|
||||
enabled: Boolean(sessionId),
|
||||
queryFn: () => (usePreviewData ? Promise.resolve(mockSessionScmSummaries[sessionId] ?? []) : fetchSessionScmSummary(sessionId)),
|
||||
queryFn: () =>
|
||||
usePreviewData ? Promise.resolve(mockSessionScmSummaries[sessionId] ?? []) : fetchSessionScmSummary(sessionId),
|
||||
retry: 1,
|
||||
};
|
||||
}
|
||||
|
|
|
|||
|
|
@ -286,11 +286,7 @@ export const mockWorkspaces: WorkspaceSummary[] = [
|
|||
},
|
||||
];
|
||||
|
||||
const prSummary = (
|
||||
sessionId: string,
|
||||
number: number,
|
||||
overrides: Partial<SessionPRSummary> = {},
|
||||
): SessionPRSummary => {
|
||||
const prSummary = (sessionId: string, number: number, overrides: Partial<SessionPRSummary> = {}): SessionPRSummary => {
|
||||
const session = mockWorkspaces.flatMap((workspace) => workspace.sessions).find((item) => item.id === sessionId);
|
||||
const facts = session?.prs.find((item) => item.number === number);
|
||||
const url = facts?.url ?? `https://github.com/me/${session?.workspaceName ?? "preview"}/pull/${number}`;
|
||||
|
|
|
|||
|
|
@ -292,7 +292,9 @@ describe("prSummaryParts", () => {
|
|||
});
|
||||
|
||||
it("puts draft readiness under Review", () => {
|
||||
const parts = prSummaryParts(summary({ state: "draft", review: { decision: "none", hasUnresolvedHumanComments: false, unresolvedBy: [] } }));
|
||||
const parts = prSummaryParts(
|
||||
summary({ state: "draft", review: { decision: "none", hasUnresolvedHumanComments: false, unresolvedBy: [] } }),
|
||||
);
|
||||
|
||||
expect(parts.find((part) => part.key === "review")).toMatchObject({
|
||||
status: "None",
|
||||
|
|
|
|||
|
|
@ -374,10 +374,7 @@ function formatLineDelta(additions: number, deletions: number): string | undefin
|
|||
return parts.length > 0 ? parts.join(" ") : undefined;
|
||||
}
|
||||
|
||||
function mergeAttentionLinks(
|
||||
pr: SessionPRSummary,
|
||||
kind: "merge_conflict" | "merge_blocked",
|
||||
): PRSummaryLink[] {
|
||||
function mergeAttentionLinks(pr: SessionPRSummary, kind: "merge_conflict" | "merge_blocked"): PRSummaryLink[] {
|
||||
const href =
|
||||
kind === "merge_conflict" ? mergeConflictUrl(pr) : pr.mergeability.prUrl || pr.htmlUrl || pr.url || undefined;
|
||||
const fileLinks = (pr.mergeability.conflictFiles ?? []).slice(0, 3).map((file) => ({
|
||||
|
|
|
|||
Loading…
Reference in New Issue