Skip to content

Conversation

@abrarsheikh
Copy link
Contributor

@abrarsheikh abrarsheikh commented May 30, 2025

The additional check on read is redundant because the writer of metrics already ensures exclusivity.

Had to increase timeout on 2 tests. I dont know why it was failing. Best guess is that its a timing issue

@abrarsheikh abrarsheikh added the go add ONLY when ready to merge, run all tests label May 30, 2025
@zcin zcin enabled auto-merge (squash) May 30, 2025 22:37
Signed-off-by: abrar <[email protected]>
@github-actions github-actions bot disabled auto-merge May 31, 2025 00:34
Signed-off-by: abrar <[email protected]>
@zcin zcin merged commit 4267914 into master Jun 2, 2025
5 checks passed
@zcin zcin deleted the SERVE-622-abrar-auto branch June 2, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants