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

Pin [email protected] #80

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Conversation

Americas
Copy link
Contributor

@Americas Americas commented Nov 6, 2024

Version 0.6.10 brought about a bug that makes traverse not work correctly on arrays when the option trim is true. This PR just pins the version of traverse to 0.6.9 since it is currently works for our purposes (the remove() has a bug, but we don't use it in this package).

Note: we can kind of fix the bug here, since the delete() call is where the bug starts, if we skip it and just do the delete directly, the code works with version 0.6.10 of traverse.

@Americas Americas requested a review from ricardobcl November 6, 2024 18:11
@Americas Americas self-assigned this Nov 6, 2024
@ricardobcl ricardobcl merged commit e01eed3 into master Nov 7, 2024
2 checks passed
@ricardobcl ricardobcl deleted the support/pin-traverse-package branch November 7, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants