Skip to content

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Oct 23, 2024

Fix the test muting on the test for grapheme clusters - it should only allow the test if we're on the 20+ jvm.

Closes #114536

Fix the test muting on the test for grapheme clusters - it should only
allow the test if we're on the 20+ jvm.

Closes elastic#114536
@nik9000 nik9000 added backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v9.0.0 labels Oct 23, 2024
@elasticsearchmachine elasticsearchmachine merged commit f2b146e into elastic:main Oct 23, 2024
16 checks passed
@nik9000 nik9000 deleted the esql_grapheme_cluster_reverse_main branch October 23, 2024 21:38
davidkyle pushed a commit to davidkyle/elasticsearch that referenced this pull request Oct 24, 2024
Fix the test muting on the test for grapheme clusters - it should only
allow the test if we're on the 20+ jvm.

Closes elastic#114536
georgewallace pushed a commit to georgewallace/elasticsearch that referenced this pull request Oct 25, 2024
Fix the test muting on the test for grapheme clusters - it should only
allow the test if we're on the 20+ jvm.

Closes elastic#114536
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Nov 4, 2024
Fix the test muting on the test for grapheme clusters - it should only
allow the test if we're on the 20+ jvm.

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

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] EsqlSpecIT test {string.ReverseGraphemeClusters SYNC} failing

2 participants