-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fix failing testSynonymsAnalysis as part of renaming #455
Conversation
…sTests.testSynonymsAnalysis due to renamingwq! Signed-off-by: Harold Wang <[email protected]>
✅ Gradle Wrapper Validation success c7fb0e8 |
❌ DCO Check Failed c7fb0e8 |
❌ DCO Check Failed a0718d2d7f011398b845994233dbfbcad9c71b98 |
✅ Gradle Wrapper Validation success a0718d2d7f011398b845994233dbfbcad9c71b98 |
a0718d2
to
3db36c9
Compare
✅ Gradle Precommit success c7fb0e8 |
✅ Gradle Precommit success a0718d2d7f011398b845994233dbfbcad9c71b98 |
❌ DCO Check Failed 3db36c98142f34eca92f4bb1cfa01caaedd17720 |
✅ Gradle Wrapper Validation success 3db36c98142f34eca92f4bb1cfa01caaedd17720 |
Signed-off-by: Harold Wang <[email protected]>
3db36c9
to
78ae587
Compare
❌ DCO Check Failed 78ae587 |
✅ Gradle Wrapper Validation success 78ae587 |
✅ Gradle Precommit success 3db36c98142f34eca92f4bb1cfa01caaedd17720 |
✅ Gradle Precommit success 78ae587 |
replaced by #456 |
This commit fixes the following two failing yaml tests " org.opensearch.analysis.common.SynonymsAnalysisTests.testSynonymsAnalysis
which are failing as a result of renaming.
Issue #441
Signed-off-by: Harold Wang [email protected]