Skip to content

Use eTags in github requests#1

Open
adamzip wants to merge 4 commits intosource-manifest-cachingfrom
github-etags-with-redis-delegate
Open

Use eTags in github requests#1
adamzip wants to merge 4 commits intosource-manifest-cachingfrom
github-etags-with-redis-delegate

Conversation

@adamzip
Copy link
Owner

@adamzip adamzip commented Jul 21, 2025

This PR is based on another ongoing PR in arcade-services

adamzip added 4 commits July 21, 2025 01:58
TargetBranchCommitSha was erroneously called "target branch" because it
represents the branch in the codeflow that belongs to the `target` repo.
Howevever, in practice it is actually the PR Head branch. Therefore it
is now renamed to HeadBranchCommitSha.
Furthermore, setting HeadBRanchCommitSha is fixed in the Azdo client.
@adamzip adamzip force-pushed the github-etags-with-redis-delegate branch from de72825 to 643db41 Compare July 21, 2025 01:23
Copy link
Owner Author

@adamzip adamzip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test

adamzip added a commit that referenced this pull request Oct 22, 2025
adamzip added a commit that referenced this pull request Oct 23, 2025
adamzip added a commit that referenced this pull request Oct 31, 2025
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