[IGNORE FOR NOW] Enforce type export/import when possible

[gsoldevila]

Summary

The PR adds an ESLINT rule that will enforce export type and import type whenever the referenced member is a type.

The goal is to find out if this can bring substantial improvements in:

• browser-side bundle sizes
• jest execution times (reducing total amount of code loaded, thus saving parsing time)

If that's the case, we'll likely have to break this down into smaller PRs, easier to pass through.

I did not make it a DRAFT PR to allow --fix from CI. You can safely ignore this PR for now 🙏🏼

[elasticmachine]

Pinging @elastic/kibana-core (Team:Core)

[elasticmachine]

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

[github-actions]

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

• /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
• run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

[gsoldevila]

/ci

[elasticmachine]

💔 Build Failed

• Buildkite Build
• Commit: d690b75

Failed CI Steps

• Checks
• Linting
• Linting (with types)

History

• 💔 Build #269580 failed a14548b
• 💔 Build #269342 failed f091fd1
• 💔 Build #269297 failed 3b546a9
• 💔 Build #269232 failed 86a9077

[mattkime]

IMO this would be a much smoother transition if it was done a few modules at a time instead of in a monster PR. Generally speaking, we should work in smaller PRs when possible. Its difficult for me to simply see the changes relevant to my team.

[dgieselaar]

IMO this would be a much smoother transition if it was done a few modules at a time instead of in a monster PR. Generally speaking, we should work in smaller PRs when possible. Its difficult for me to simply see the changes relevant to my team.

The hint is in the title!

[gsoldevila]

A more recent attempt has been performed with #212221