FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix DT result summaries by jakebailey · Pull Request #1346 · microsoft/DefinitelyTyped-tools · GitHub

Repository navigation

Fix DT result summaries - #1346

Open
Jake Bailey (jakebailey) wants to merge 4 commits into
microsoft:mainfrom
jakebailey:dt-result-summary-package-names
Open

Jake Bailey (jakebailey) wants to merge 4 commits into
microsoft:mainfrom
jakebailey:dt-result-summary-package-names

Conversation

Jake Bailey (jakebailey) commented Oct 5, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

microsoft/TypeScript#64604 (comment) is annoying because the expando titles are all the same.

While here, add retries to the Go download.

Collapsed result sections otherwise look identical, making it hard to
find a package without expanding every report.
Jake Bailey (jakebailey) changed the title Show package names in DT result summaries Fix DT result summaries Oct 5, 2026
The first diagnostic was left unlinked because its Error prefix was
mistaken for part of the filename. Absolute checkout paths also made
linked locations noisy and specific to the CI agent.

Link the first diagnostic and display locations relative to the DT
repository so the reports are easier to navigate.
A transient manifest download failure on one shard can fail the entire
DT run. Retry Go installation as we already do for repository checkouts.

This branch has not been deployed

No deployments
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL