[kibanaAppServices] Remove export* syntax from plugin index files #110897
Labels
chore
impact:low
Addressing this issue will have a low level of impact on the quality/strength of our product.
loe:small
Small Level of Effort
See #57370 for reasoning, but the gist is that
export *
usage leads to unexpected and excessive exports from the plugin, leading the public API to grow and increasing the cost of maintaining backwards compatibility.#109357 implements an ESLint rule to check for this that also assists in fixing the rule, though the auto-fix is imperfect and will require some manual review before committing.
Please remove the
/* eslint-disable @kbn/eslint/no_export_all */
comment(s) added in #109357 from the relevant files, validate the auto-fix, remove unnecessary exports, and then submit a PR. Thank you!Hint: You can search for the URL of this issue to find specific occurrences
For each plugin:
server/index.ts
,public/index.ts
andcommon/index.ts
folders for each plugin below, repalce*
exports by a named export for each thing.Plugins to check:
eslint-rule-no-export-all
#109905 remove star export in bfetch plugin #121806eslint-rule-no-export-all
#109904eslint-rule-no-export-all
#109902eslint-rule-no-export-all
#109900The text was updated successfully, but these errors were encountered: