-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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 ascendants check #206080
🌊 Fix ascendants check #206080
Conversation
💚 Build Succeeded
Metrics [docs]
|
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.
LGTM 👌
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
Starting backport for target branches: 8.x |
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
Starting backport for target branches: 8.x |
The id->name refactoring didn't catch the check for ascendants which was still looking for id. This PR fixes this (cherry picked from commit 23c958c)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.x`: - [🌊 Fix ascendants check (#206080)](#206080) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Joe Reuter","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-10T15:48:13Z","message":"🌊 Fix ascendants check (#206080)\n\nThe id->name refactoring didn't catch the check for ascendants which was\r\nstill looking for id. This PR fixes this","sha":"23c958c1a1330f29d9d4b55db305db81a069106b","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","backport:version","v8.18.0","Feature:Streams"],"title":"🌊 Fix ascendants check","number":206080,"url":"https://github.com/elastic/kibana/pull/206080","mergeCommit":{"message":"🌊 Fix ascendants check (#206080)\n\nThe id->name refactoring didn't catch the check for ascendants which was\r\nstill looking for id. This PR fixes this","sha":"23c958c1a1330f29d9d4b55db305db81a069106b"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/206080","number":206080,"mergeCommit":{"message":"🌊 Fix ascendants check (#206080)\n\nThe id->name refactoring didn't catch the check for ascendants which was\r\nstill looking for id. This PR fixes this","sha":"23c958c1a1330f29d9d4b55db305db81a069106b"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Joe Reuter <[email protected]>
The id->name refactoring didn't catch the check for ascendants which was still looking for id. This PR fixes this
The id->name refactoring didn't catch the check for ascendants which was still looking for id. This PR fixes this