Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: add stripTrailingZero function #116

Merged
merged 2 commits into from
Jan 20, 2024
Merged

Conversation

yogski
Copy link
Contributor

@yogski yogski commented Oct 1, 2023

#82

Changes:

  • Moved existing trailZero() as new function stripTrailingZero() for removing both prefix and suffix zeroes.
  • Added test case with 100% coverage for specified function.
  • Update readme, validated doctoc

Please let me know what you think, especially the intended behavior for removing zeroes both in front and back, as well as the function name if there's better idea.

Good day.

@royNiladri royNiladri merged commit beff862 into royNiladri:master Jan 20, 2024
4 checks passed
@royNiladri
Copy link
Owner

Thanks @yogski for the contribution.. Hope to roll out a build with this soon!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants