klaudiosinani / doublie Goto Github PK

Description

The following issue will act as a progress tracking thread, regarding the implementation of the exposed Linear class methods:

Methods

• Linear#append(...values)
• Linear#filter(fn)
• Linear#forEach(fn)
• Linear#get(index)
• Linear#join(separator)
• Linear#reduce(fn, accumulator)
• Linear#map(fn)
• Linear#node(index)
• Linear#prepend(...values)
• Linear#remove(index)
• Linear#reverse()
• Linear#set({value, index})
• Linear#toArray()
• Linear#reduceRight(fn, accumulator)
• Linear#insert({value, index})

Description

The following issue will act as a progress tracking thread, regarding the implementation of the exposed Circular class methods:

Methods

• Circular#append(...values)
• Circular#filter(fn)
• Circular#forEach(fn)
• Circular#get(index)
• Circular#join(separator)
• Circular#reduce(fn, accumulator)
• Circular#map(fn)
• Circular#node(index)
• Circular#prepend(...values)
• Circular#remove(index)
• Circular#reverse()
• Circular#set({value, index})
• Circular#toArray()
• Circular#reduceRight(fn, accumulator)
• Circular#insert({value, index})
• Circular#toLinear()

Description

Add TypeScript support by declaring types for the exported Doublie API methods.

Please visit the contributing guidelines to learn more on how to translate Doublie's documentation into more languages.

Tons of thank you to everyone for your help and support! ❤️

Is your feature request related to a problem? Please describe.
In the typescript definitions, the node properties .next and .prev are of type Instance<T> | null, even for circular doubly linked list nodes (which uses the same type), meaning that using it in strict mode involves a lot of null checks that this type should not need.

Describe the solution you'd like
The node properties in this case should only be of type Instance<T>.

Additional context
n/a