Skip to content

getPullRequestEditedDiffRanges: use GitHub API #2679

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cklin merged 8 commits into from
Jan 14, 2025
Merged

getPullRequestEditedDiffRanges: use GitHub API #2679

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
68378a3
getPullRequestEditedDiffRanges: use GitHub API
cklin Jan 13, 2025
68b1b4e
getPullRequestEditedDiffRanges: compute diff ranges
cklin Jan 13, 2025
63d1f4f
getPullRequestEditedDiffRanges: check the number of changed files
cklin Jan 13, 2025
d4c56bb
getDiffRanges: improve handling for missing patch
cklin Jan 9, 2025
7848967
getDiffRanges: add unit tests
cklin Jan 13, 2025
52315a9
setupDiffInformedQueryRun: log setup failure
cklin Jan 9, 2025
3548ff5
writeDiffRangeDataExtensionPack: escape special characters in filenames
cklin Jan 10, 2025
ae19466
build: refresh js files
cklin Jan 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
writeDiffRangeDataExtensionPack: escape special characters in filenames
  • Loading branch information
@cklin
cklin committed Jan 13, 2025
commit 3548ff54b5e54db237049bac959e13f48db15a76
6 changes: 5 additions & 1 deletion src/analyze.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -512,7 +512,11 @@ extensions:
let data = ranges
.map(
(range) =>
` - ["${range.path}", ${range.startLine}, ${range.endLine}]\n`,
// Using yaml.dump() with `forceQuotes: true` ensures that all special
// characters are escaped, and that the path is always rendered as a
// quoted string on a single line.
` - [${yaml.dump(range.path, { forceQuotes: true }).trim()}, ` +
`${range.startLine}, ${range.endLine}]\n`,
)
.join("");
if (!data) {
Expand Down