Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

fix: removeOne(-1) from the end#55

Open
teidesu wants to merge 1 commit into
invertase:masterfrom
teidesu:fix-removeone
Open

fix: removeOne(-1) from the end#55
teidesu wants to merge 1 commit into
invertase:masterfrom
teidesu:fix-removeone

Conversation

@teidesu

@teidesu teidesu commented Sep 15, 2024

Copy link
Copy Markdown

mcve

const b = new Denque([1, 2, 3, 4, 5, 6, 7, 8]);
console.log(b.removeOne(-1));
console.log(b.toArray());

expected result

8
[1, 2, 3, 4, 5, 6, 7]

actual result

8
[2, 3, 4, 5, 6, 7, undefined]

@docs-page

docs-page Bot commented Sep 15, 2024

Copy link
Copy Markdown

To view this pull requests documentation preview, visit the following URL:

docs.page/invertase/denque~55

Documentation is deployed and generated using docs.page.

@CLAassistant

CLAassistant commented Sep 15, 2024

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants