New TypeScript definition of 'patch' uses Buffer as a generic about node-diff3 HOT 3 CLOSED

Ah, yeah I don't actually use TypeScript at all. I basically just blindly accept any pull request involving the indes.d.ts file 👍

from node-diff3.

Ok, I can see the issue, this is the actual line from index.d.ts (not sure why the messages show it as what I put in the original post instead):

export function patch<T>(buffer: T[], patch: IPatchRes<T>[]): Buffer<T>[];

"Buffer" is indeed an attempt to use Buffer as a generic. If the function truly returns an arrray of binary Buffer objects, the correct export would be:

export function patch<T>(buffer: T[], patch: IPatchRes<T>[]): Buffer[];

If the function returns an array of type T, which seems more consistent with the other functions, it would be

export function patch<T>(buffer: T[], patch: IPatchRes<T>[]): T[];

Happy to make the pull request if you want

from node-diff3.

Thanks, I went with T[] as the return type.. Just published v2.1.2 with this fix 👍

from node-diff3.