Skip to content

Windows OS encoding issue #343

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

Merged
merged 1 commit into from
Oct 14, 2021
Merged

Conversation

jrinder42
Copy link
Contributor

Type of PR

Bug fix - Issue #342

Description

Explicitly add the encoding type when creating the long description in the setup.py script, otherwise running tox throws an error in Windows

@curquiza curquiza requested a review from alallema October 14, 2021 12:27
@curquiza curquiza linked an issue Oct 14, 2021 that may be closed by this pull request
@alallema alallema changed the title Fix: #342 - Windows OS encoding issue Windows OS encoding issue Oct 14, 2021
@alallema alallema added the skip-changelog The PR will not appear in the release changelogs label Oct 14, 2021
Copy link
Contributor

@alallema alallema left a comment

Choose a reason for hiding this comment

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

Thanks for this PR @jrinder42 and for raising the associate issue!
LGTM! 🎉

Thank you for contributing to MeiliSearch. If you are participating in Hacktoberfest, and you would like to receive a small gift from MeiliSearch too, please complete this form.

@alallema
Copy link
Contributor

bors merge

@bors
Copy link
Contributor

bors bot commented Oct 14, 2021

@bors bors bot merged commit 3e9a438 into meilisearch:main Oct 14, 2021
@jrinder42 jrinder42 deleted the setup-windows-fix branch October 14, 2021 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog The PR will not appear in the release changelogs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows issue with running setup.py - long description
2 participants