Skip to content

Conversation

@simonswine
Copy link
Contributor

@simonswine simonswine commented Feb 19, 2025

I am a bit conflicted, as we are seeing a fair bit of test failures. This change will obvisouly help getting PRs merged quicker, but it might also lead to a downward slope in terms of test quality.

See #3923

@simonswine simonswine force-pushed the 20250219_retry-failed-tests branch from 3477d1f to 740cd0b Compare February 19, 2025 13:13
I am a bit conflicted, as we are seeing a fair bit of test failures.
This change will obvisouly help getting PRs merged quicker, but it might
also lead to a downward slope in terms of test quality.
@simonswine simonswine force-pushed the 20250219_retry-failed-tests branch from 740cd0b to cdd3658 Compare February 19, 2025 13:41
@simonswine simonswine marked this pull request as ready for review February 19, 2025 14:18
@simonswine simonswine requested a review from a team as a code owner February 19, 2025 14:18
Copy link
Collaborator

@kolesnikovae kolesnikovae left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks for adding this! I agree with your reasoning

@simonswine simonswine merged commit b6b7bd0 into grafana:main Feb 21, 2025
21 checks passed
shelldandy pushed a commit to shelldandy/pyroscope that referenced this pull request Mar 14, 2025
I am a bit conflicted, as we are seeing a fair bit of test failures.
This change will obvisouly help getting PRs merged quicker, but it might
also lead to a downward slope in terms of test quality.
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