-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution][DQD][Tech Debt] Refactor top level helpers #191233
[Security Solution][DQD][Tech Debt] Refactor top level helpers #191233
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-threat-hunting-explore (Team:Threat Hunting:Explore) |
5d6788f
to
6e8555a
Compare
6e8555a
to
e834eb5
Compare
e834eb5
to
6b3b423
Compare
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.
This is great. Much better organized than before. Thanks Karen!
LGTM
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: cc @kapral18 |
…1245) addresses #190964 Fourth in the series of PRs to address general DQD tech debt This one builds on previous 3 PRs #190970 #190978 #191233 Gist of changes: split lower level helpers into series of utils/* files each utils/ file is named after common behavior it export or works with. cleanup dead code
addresses #190964
Third in the series of PRs to address general DQD tech debt
This one builds on previous 2 PRs
#190970
#190978
Gist of changes: