Skip to content
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

[Infra] Remove unused files #181964

Open
crespocarlos opened this issue Apr 29, 2024 · 1 comment
Open

[Infra] Remove unused files #181964

crespocarlos opened this issue Apr 29, 2024 · 1 comment
Labels
Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture

Comments

@crespocarlos
Copy link
Contributor

Summary

There are some files in infra that are not used and can be removed. Some examples are:

  • x-pack/plugins/observability_solution/infra/public/utils/typed_redux.ts
  • x-pack/plugins/observability_solution/infra/public/utils/typed_react.tsx (except composeStateUpdaters function)
  • x-pack/plugins/observability_solution/infra/public/utils/state_container_devtools.ts
  • x-pack/plugins/observability_solution/infra/public/utils/loading_state (seems like this whole folder can be removed)
  • x-pack/plugins/observability_solution/infra/public/utils/cancellable_effect.ts
  • x-pack/plugins/observability_solution/infra/public/utils/dev_mode.ts (redux related functions)
  • x-pack/plugins/observability_solution/infra/public/utils/handlers.ts
  • x-pack/plugins/observability_solution/infra/public/utils/history_context.ts
  • x-pack/plugins/observability_solution/infra/public/utils/log_column_render_configuration.tsx
  • x-pack/plugins/observability_solution/infra/public/utils/logs_overview_fetchers.ts (except getLogsHasDataFetcher and getLogsOverviewDataFetcher)
  • x-pack/plugins/observability_solution/infra/public/utils/header_action_menu_provider.tsx (move to another folder - it's a context provider, not utils)

AC

  • Unused files in infra are removed
@crespocarlos crespocarlos added the Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team label Apr 29, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@smith smith added the technical debt Improvement of the software architecture and operational architecture label Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Development

No branches or pull requests

3 participants