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

normalized -> trimmed amount #198

Merged
merged 1 commit into from
Feb 26, 2024
Merged

normalized -> trimmed amount #198

merged 1 commit into from
Feb 26, 2024

Conversation

kcsongor
Copy link
Contributor

this name makes more sense, given that these amounts are not technically normalized (there's no normal form)

@kcsongor kcsongor force-pushed the normalize-to-trimmed branch 2 times, most recently from 46892ae to b5e659b Compare February 25, 2024 23:39
RahulMaganti47
RahulMaganti47 previously approved these changes Feb 25, 2024
djb15
djb15 previously approved these changes Feb 26, 2024
Copy link
Collaborator

@djb15 djb15 left a comment

Choose a reason for hiding this comment

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

Lgtm

@kcsongor kcsongor merged commit bdc8a23 into main Feb 26, 2024
4 checks passed
@kcsongor kcsongor deleted the normalize-to-trimmed branch February 26, 2024 04:39
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.

3 participants