Skip to content

[7.x] Use consistent-type-imports lint rule to autofix type-only imports (#94464)#94868

Merged
jfsiii merged 1 commit intoelastic:7.xfrom
jfsiii:backport/7.x/pr-94464
Mar 17, 2021
Merged

[7.x] Use consistent-type-imports lint rule to autofix type-only imports (#94464)#94868
jfsiii merged 1 commit intoelastic:7.xfrom
jfsiii:backport/7.x/pr-94464

Conversation

@jfsiii
Copy link
Copy Markdown
Contributor

@jfsiii jfsiii commented Mar 17, 2021

Backports the following commits to 7.x:

…94464)

## Summary

Use the [`consistent-type-imports` lint rule ](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/consistent-type-imports.md) to use `import type ...` where possible

Code-only changes like this were recently added in #92979, but it was a more manual process.  This PR adds an autofixable lint rule which will keep the changes up-to-date and Just Work ™️
# Conflicts:
#	x-pack/plugins/fleet/server/services/agents/crud.ts
#	x-pack/plugins/fleet/server/services/artifacts/artifacts.ts
#	x-pack/plugins/fleet/server/services/artifacts/mappings.ts
#	x-pack/plugins/fleet/server/services/artifacts/mocks.ts
#	x-pack/plugins/fleet/server/services/index.ts
@jfsiii jfsiii added the backport This PR is a backport of another PR label Mar 17, 2021
@jfsiii jfsiii enabled auto-merge (squash) March 17, 2021 21:20
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jfsiii jfsiii merged commit ad36e8c into elastic:7.x Mar 17, 2021
@jfsiii jfsiii deleted the backport/7.x/pr-94464 branch April 6, 2021 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants