Skip to content

fix: typo on ov #29

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

Merged
merged 2 commits into from
Sep 1, 2021
Merged

fix: typo on ov #29

merged 2 commits into from
Sep 1, 2021

Conversation

soedirgo
Copy link
Collaborator

@soedirgo soedirgo commented Sep 1, 2021

What kind of change does this PR introduce?

Closes #28.

@Danacus
Copy link

Danacus commented Sep 1, 2021

Was just about to open a PR, you beat me to it :). You should probably also fix the documentation of this function.

@soedirgo
Copy link
Collaborator Author

soedirgo commented Sep 1, 2021

Ah, thanks for opening the issue still!

@soedirgo soedirgo merged commit f6fc126 into master Sep 1, 2021
@soedirgo soedirgo deleted the fix/ov branch September 1, 2021 08:16
@Danacus
Copy link

Danacus commented Sep 1, 2021

Thanks a lot for the quick response!

# 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.

Incorrect implementation of ov
2 participants