import/export: false positives for typescript overloads

[Philipp91]

Minimal example to reproduce (not a sensible use case for overloads, but anyway):

export function consume(val: number): void;
export function consume(val: string): void;
export function consume(val: number | string): void {
    // Do something.
}

All three consume will be annotated with error Multiple exports of name 'consume' import/export.

[ljharb]

Actually do you need all the export keywords? I’d expect the first two to not need it.

[Philipp91]

Sorry, I should have mentioned this. Leaving them off results in a TypeScript compiler error: TS2383: Overload signatures must all be exported or non-exported.

[leethree]

I'm encountering the same issue.

[matt-tingen]

Here's a workaround:

interface Consume {
  (val: number): void;
  (val: string): void;
}
export const consume: Consume = (val: number | string) => {
  // ...
}

[jose-nunez]

The interface workaround works for me and looks nicer than exporting all the overloads.
Thanks!

[tylerthehaas]

The work around is good and nice that there is an alternate syntax but the plugin should be able to handle all the features of typescript

[ljharb]

Agreed; a PR to fix it is welcome.