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

apply bug fixes from upstream #2

Merged
merged 2 commits into from
May 30, 2022
Merged

apply bug fixes from upstream #2

merged 2 commits into from
May 30, 2022

Conversation

ksco
Copy link
Contributor

@ksco ksco commented May 30, 2022

This commit applies bug fixes from the following upstream commits:

  1. Turbo87/utm@eeed1f1
  2. Turbo87/utm@17e570c

@icholy
Copy link
Owner

icholy commented May 30, 2022

Thanks, do you mind copying over the test cases from the linked commits?

@ksco
Copy link
Contributor Author

ksco commented May 30, 2022

Yes, I should definitely include the test as well.

This commit applies bug fixes from the following upstream commits:
1. Turbo87/utm@eeed1f1
2. Turbo87/utm@17e570c
@ksco
Copy link
Contributor Author

ksco commented May 30, 2022

Through testing, I found that Go's math.Mod and Python's % operator are not equivalent. Thanks for the advice!

@icholy icholy merged commit 2276eea into icholy:master May 30, 2022
@icholy
Copy link
Owner

icholy commented May 30, 2022

Thanks, I've tagged this as v1.0.1

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