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.

Sign up for GitHub

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

Jump to bottom

[Infra + Logs UI] Remove and fix linting rule exceptions #49610

Closed
weltenwort opened this issue Oct 29, 2019 · 2 comments
Closed

[Infra + Logs UI] Remove and fix linting rule exceptions #49610

weltenwort opened this issue Oct 29, 2019 · 2 comments
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services technical debt Improvement of the software architecture and operational architecture

Comments

@weltenwort
Copy link
Member

weltenwort commented Oct 29, 2019 •
edited by sgrodzicki
Loading

In #49244 a set of linting rule have been enabled for .ts? files. But since the infra plugin requires non-trivial cleanup to satisfy all the rules, they have been selectively disabled. We should re-enable the rules and fix the rule violations.
@weltenwort weltenwort added Feature:Metrics UI Metrics UI feature technical debt Improvement of the software architecture and operational architecture Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services technical_debt labels Oct 29, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@weltenwort
Copy link
Member Author

duplicate of #49563

@weltenwort weltenwort mentioned this issue Nov 20, 2019
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature:Logs UI Logs UI feature Feature:Metrics UI Metrics UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services technical debt Improvement of the software architecture and operational architecture
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@weltenwort @elasticmachine