Skip to content

Add some comment for rr testcase#515

Closed
rongyi wants to merge 1 commit intocloudflare:mainfrom
rongyi:main
Closed

Add some comment for rr testcase#515
rongyi wants to merge 1 commit intocloudflare:mainfrom
rongyi:main

Conversation

@rongyi
Copy link
Contributor

@rongyi rongyi commented Jan 17, 2025

Add some comment for someone just like me who confused the order for a little bit, and I do think the hash testcase is too predefined in this file, e.g. the random hash("test").

@gumpt gumpt added the documentation Improvements or additions to documentation label Jan 17, 2025
@eaufavor eaufavor added the WIP We are working on this feature internally label Jan 21, 2025
@andrewhavck andrewhavck added Accepted This change is accepted by us and merged to our internal repo and removed WIP We are working on this feature internally labels Jan 31, 2025
eaufavor pushed a commit that referenced this pull request Feb 14, 2025
Includes-commit: 7e8b7fd
Replicated-from: #515
eaufavor pushed a commit that referenced this pull request Feb 14, 2025
Includes-commit: 7e8b7fd
Replicated-from: #515
xxargonaut added a commit to xxargonaut/pingora that referenced this pull request Feb 19, 2025
Includes-commit: 7e8b7fd2842bed6031e3027adc5f3113a41c53ab
Replicated-from: cloudflare/pingora#515
xxargonaut added a commit to xxargonaut/pingora that referenced this pull request Feb 19, 2025
Includes-commit: 7e8b7fd2842bed6031e3027adc5f3113a41c53ab
Replicated-from: cloudflare/pingora#515
@drcaramelsyrup
Copy link
Collaborator

Thanks! this was committed in 043b69a.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted This change is accepted by us and merged to our internal repo documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants