-
Notifications
You must be signed in to change notification settings - Fork 936
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
[Backport 2.x] [Auto Suggest] Fix Grammar Changes #9313
Conversation
* sql more field name tests Signed-off-by: Paul Sebastian <[email protected]> * undo search command change w/ updated logic, generalize rerun flag to work with any lang and any number of rules Signed-off-by: Paul Sebastian <[email protected]> * beginning of updating combined results properly Signed-off-by: Paul Sebastian <[email protected]> * Changeset file for PR #9120 created/updated * cleaner non-recursive system to go through rerunWithout rule list and a general nonspecific combine function Signed-off-by: Paul Sebastian <[email protected]> * revert qualified name grammar Signed-off-by: Paul Sebastian <[email protected]> * revert from and table source grammars Signed-off-by: Paul Sebastian <[email protected]> * fix ppl field name issue Signed-off-by: Paul Sebastian <[email protected]> * remove ppl agg func snippets and update ppl field name test Signed-off-by: Paul Sebastian <[email protected]> * leave a space for sql agg func Signed-off-by: Paul Sebastian <[email protected]> * ppl autocomplete more pref. rules test Signed-off-by: Paul Sebastian <[email protected]> * remove backticks from ppl field name to match for value completion Signed-off-by: Paul Sebastian <[email protected]> * add comment explaining result combination Signed-off-by: Paul Sebastian <[email protected]> --------- Signed-off-by: Paul Sebastian <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit d227bc5) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
❌ Invalid Changelog HeadingThe '## Changelog' heading in your PR description is either missing or malformed. Please make sure that your PR description includes a '## Changelog' heading with proper spelling, capitalization, spacing, and Markdown syntax. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #9313 +/- ##
==========================================
+ Coverage 61.77% 61.79% +0.02%
==========================================
Files 3815 3815
Lines 91715 91843 +128
Branches 14513 14539 +26
==========================================
+ Hits 56653 56754 +101
+ Misses 31441 31438 -3
- Partials 3621 3651 +30
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Cypress tests are passing, it's currently failing on artifact uploads. |
Backport d227bc5 from #9120.