-
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
Remove /src/legacy #95510
Remove /src/legacy #95510
Changes from all commits
264e77c
60effbc
dac4ac0
3427c1a
cff9b0b
46260c8
5a9ff60
4a9bd3a
9628609
510caae
8d5cadf
7daef0a
871e4c3
dd02cfa
5148dd4
f4f9ad0
3443fa1
c1eab04
4b5df60
fa8da2b
fbf2618
d1d64e1
f51f0cd
d4a5c97
ac7baa3
b82e0c0
6a45836
1f29cb6
89bf3d6
315a8b2
69a9d8c
04e90cc
436e094
af6c32d
a057a60
a49f2e1
2445460
4541ab1
d75d880
b16ebf3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
All that remains in this file is this
Public / Server
export. The file seems unused in the repo, but I wasn't sure if I should remove it.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.
I believe this used for the
import { CoreSetup } from 'kibana/server'
TS shortcuts in the repo. I believe we'll need to migrate away from those for Bazel anyways, but that should be done separate from this PR.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.
Err scratch that, I think the generated types are used to power that. So yeah, not sure if we need this strictly anymore either.
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.
they are configured with TS paths
kibana/tsconfig.base.json
Lines 7 to 8 in 50313f7
I didn't manage to find any places relying on it.
@elastic/kibana-operations maybe you know? If no, let's remove it