Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bottom for source size == 1 #13

Merged
merged 3 commits into from
Nov 18, 2021
Merged

Conversation

dgilperez
Copy link
Collaborator

Primary

  1. Fix bug in bottom when the source size is exactly 1 and the limit is longer than that - it was removing that element when it should not.

Checklist

Copy link
Collaborator

@andreidmt andreidmt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Nice to "see" you and thank you for the PR

PS: Im working on adding TS types :)

@andreidmt andreidmt merged commit 466cae6 into asd-xiv:master Nov 18, 2021
andreidmt pushed a commit that referenced this pull request Nov 18, 2021
## [7.5.1](v7.5.0...v7.5.1) (2021-11-18)

### Bug Fixes

* **bottom:** issue returning an empty array when source size is exactly 1 and limit is grater ([#13](#13)) ([466cae6](466cae6))
@andreidmt
Copy link
Collaborator

🎉 This PR is included in version 7.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@dgilperez
Copy link
Collaborator Author

Thanks a lot for the speedy merge! Looking forward to that typing as well :)

@dgilperez dgilperez deleted the fix/bottom branch November 21, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants