Skip to content

DEVPROD-29442 Get raw patch rather than diff for GitHub patches#9895

Merged
ZackarySantana merged 2 commits intoevergreen-ci:mainfrom
ZackarySantana:DEVPROD-29442
Mar 5, 2026
Merged

DEVPROD-29442 Get raw patch rather than diff for GitHub patches#9895
ZackarySantana merged 2 commits intoevergreen-ci:mainfrom
ZackarySantana:DEVPROD-29442

Conversation

@ZackarySantana
Copy link
Copy Markdown
Contributor

@ZackarySantana ZackarySantana commented Mar 5, 2026

DEVPROD-29442

Description

When we apply the diff for a patch, we expect it to have the full contents (e.g. binary files). This changes what we fetch from github from just diff, which is intended for viewing, to patch, which is intended for full changes.

If you want to see the difference for a png:

Testing

I added a test, running it on main fails. Running it with these changes passes.

I tested it in staging by running this specific scenario with and without the changes.

This PR contains the example.

@ZackarySantana ZackarySantana self-assigned this Mar 5, 2026
@ZackarySantana ZackarySantana requested a review from a team March 5, 2026 21:28
@ZackarySantana ZackarySantana changed the title DEVPROD-29442 Get raw patch rather than diff DEVPROD-29442 Get raw patch rather than diff for GitHub patches Mar 5, 2026
@ZackarySantana ZackarySantana merged commit 7da525f into evergreen-ci:main Mar 5, 2026
10 of 12 checks passed
@ZackarySantana ZackarySantana deleted the DEVPROD-29442 branch March 5, 2026 22:23
ZackarySantana added a commit that referenced this pull request Mar 9, 2026
malikchaya2 pushed a commit to malikchaya2/evergreen that referenced this pull request Mar 18, 2026
malikchaya2 pushed a commit to malikchaya2/evergreen that referenced this pull request Mar 18, 2026
malikchaya2 pushed a commit to malikchaya2/evergreen that referenced this pull request Mar 18, 2026
malikchaya2 pushed a commit to malikchaya2/evergreen that referenced this pull request Mar 18, 2026
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.

2 participants