Skip to content

[ES|QL] Clean up trigger chars#218023

Merged
drewdaemon merged 26 commits into
elastic:mainfrom
drewdaemon:clean-up-trigger-chars
Apr 14, 2025
Merged

[ES|QL] Clean up trigger chars#218023
drewdaemon merged 26 commits into
elastic:mainfrom
drewdaemon:clean-up-trigger-chars

Conversation

@drewdaemon
Copy link
Copy Markdown
Contributor

@drewdaemon drewdaemon commented Apr 11, 2025

Summary

There were several trigger characters on our completion item provider that weren't doing anything. This removes them!

@drewdaemon drewdaemon added Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana t// backport:version Backport to applied version labels v9.1.0 v8.19.0 release_note:skip Skip the PR/issue when compiling release notes labels Apr 11, 2025
@drewdaemon drewdaemon marked this pull request as ready for review April 12, 2025 00:49
@drewdaemon drewdaemon requested a review from a team as a code owner April 12, 2025 00:49
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@drewdaemon
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kbnUiSharedDeps-srcJs 3.6MB 3.6MB -11.0B

History

@drewdaemon drewdaemon merged commit 7dcb22b into elastic:main Apr 14, 2025
9 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/14453908141

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 14, 2025
## Summary

There were several trigger characters on our completion item provider
that weren't doing anything. This removes them!

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 7dcb22b)
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Apr 14, 2025
# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Clean up trigger chars
(#218023)](#218023)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Drew
Tate","email":"drew.tate@elastic.co"},"sourceCommit":{"committedDate":"2025-04-14T19:19:47Z","message":"[ES|QL]
Clean up trigger chars (#218023)\n\n## Summary\n\nThere were several
trigger characters on our completion item provider\nthat weren't doing
anything. This removes them!\n\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"7dcb22bdec595c618666ae7bc3e218539a3f24c1","branchLabelMapping":{"^v9.1.0$":"main","^v8.19.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:ES|QL","Team:ESQL","backport:version","v9.1.0","v8.19.0"],"title":"[ES|QL]
Clean up trigger
chars","number":218023,"url":"https://github.com/elastic/kibana/pull/218023","mergeCommit":{"message":"[ES|QL]
Clean up trigger chars (#218023)\n\n## Summary\n\nThere were several
trigger characters on our completion item provider\nthat weren't doing
anything. This removes them!\n\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"7dcb22bdec595c618666ae7bc3e218539a3f24c1"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.1.0","branchLabelMappingKey":"^v9.1.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/218023","number":218023,"mergeCommit":{"message":"[ES|QL]
Clean up trigger chars (#218023)\n\n## Summary\n\nThere were several
trigger characters on our completion item provider\nthat weren't doing
anything. This removes them!\n\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"7dcb22bdec595c618666ae7bc3e218539a3f24c1"}},{"branch":"8.x","label":"v8.19.0","branchLabelMappingKey":"^v8.19.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Drew Tate <drew.tate@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels Feature:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana t// v8.19.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants