-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Remove awaits for closed issues #132306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove awaits for closed issues #132306
Conversation
9dfe06b to
a01521c
Compare
7e1d6dc to
b2c235d
Compare
9d6d207 to
36b8b20
Compare
muted-tests.yml
Outdated
| method: test {csv-spec:mv_percentile.FromIndexPercentile} | ||
| issue: https://github.com/elastic/elasticsearch/issues/132846 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems weird, shouldn't there be another class line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes seems like merge conflict resolved badly, will fix. Thanks!
|
Pinging @elastic/es-core-infra (Team:Core/Infra) |
Fill in some issues
More issues
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 thank you for addressing this debt!
* Remove awaits for closed issues
Remove
@AwaitFixtags for issues that are closed. If the test is broken, this should be surfaced, there's no point to keep it around muted since the issue is not going to be looked on when it's closed.