-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
refact(NA): remove set loop from upgrade assistant usage collector #63977
refact(NA): remove set loop from upgrade assistant usage collector #63977
Conversation
Pinging @elastic/kibana-operations (Team:Operations) |
x-pack/plugins/upgrade_assistant/server/lib/telemetry/usage_collector.ts
Outdated
Show resolved
Hide resolved
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.
@mistic Thanks for adding this logic, would you mind describing in the PR what issue it was causing?
Also, I really would prefer to not use a dependency for this functionality, especially because there is native functionality Object.prototype.hasOwnProperty
which does exactly what we want here. Use of lodash in general should be scaled back where possible.
…from-ua-collector
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
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.
Happy with the updated changes 👍.
…lastic#63977) * refact(NA): use default telemetry obj in the set loop * chore(NA): fix typecheck * fix(NA): test run with lodash has * chore(NA): fix older lodash has * chore(NA): remove lodash has usage
…lastic#63977) * refact(NA): use default telemetry obj in the set loop * chore(NA): fix typecheck * fix(NA): test run with lodash has * chore(NA): fix older lodash has * chore(NA): remove lodash has usage
…lastic#63977) * refact(NA): use default telemetry obj in the set loop * chore(NA): fix typecheck * fix(NA): test run with lodash has * chore(NA): fix older lodash has * chore(NA): remove lodash has usage
Small refactor to remove set loop from the upgrade assistant usage collector.