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

Unused functions within the helpers package have been removed. #2738

Closed
wants to merge 1 commit into from

Conversation

tpgus
Copy link

@tpgus tpgus commented Oct 25, 2024

  • Is this a bug fix, new functionality, or a breaking change?
  • Have read and followed the steps for preparing a pull request.

@smallsaucepan
Copy link
Member

Thanks @tpgus. We might need to wait until a major release to clean this up though. Although the documentation says @private we do still export the function (admittedly contradictory). Someone may be importing it so I feel we need to keep it available for now.

@tpgus
Copy link
Author

tpgus commented Oct 28, 2024

@smallsaucepan.

I understand. Thank you for your reply.

@smallsaucepan
Copy link
Member

Thanks @tpgus. Will take another look when we're preparing to release Turf 8.

@tpgus tpgus closed this by deleting the head repository Nov 9, 2024
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