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

UnicodeDecodeError for README.md when installing #176

Closed
github-actions bot opened this issue Jul 7, 2024 · 3 comments
Closed

UnicodeDecodeError for README.md when installing #176

github-actions bot opened this issue Jul 7, 2024 · 3 comments

Comments

@github-actions
Copy link

github-actions bot commented Jul 7, 2024

GitHub Actions workflow tests #334 failed.

Event: schedule
Branch: main
Commit: 4f1fc6bf1bed0d489433ea428c8323d6eab1f18a

Created by jayqi/failed-build-issue-action

Copy link
Author

github-actions bot commented Jul 8, 2024

GitHub Actions workflow tests #334 failed.

Event: schedule
Branch: main
Commit: 4f1fc6bf1bed0d489433ea428c8323d6eab1f18a

Created by jayqi/failed-build-issue-action

@jayqi
Copy link
Member

jayqi commented Jul 8, 2024

It's possible there's some real error here, though I'm not sure why this didn't cause a failure in #175.

EDIT: Aha, it looks like README.md is autogenerated by CI and not actually part of the PR. So #175 changed the dependencies that populate README.md, and then this commit actually propagates those changes.


Traceback is:

Traceback (most recent call last):
  File "<string>", line 14, in <module>
  File "C:\Users\runneradmin\AppData\Local\uv\cache\builds-v0\.tmpxJp0XV\lib\site-packages\setuptools\build_meta.py", line 458, in get_requires_for_build_editable
    return self.get_requires_for_build_wheel(config_settings)
  File "C:\Users\runneradmin\AppData\Local\uv\cache\builds-v0\.tmpxJp0XV\lib\site-packages\setuptools\build_meta.py", line 327, in get_requires_for_build_wheel
    return self._get_build_requires(config_settings, requirements=[])
  File "C:\Users\runneradmin\AppData\Local\uv\cache\builds-v0\.tmpxJp0XV\lib\site-packages\setuptools\build_meta.py", line 297, in _get_build_requires
    self.run_setup()
  File "C:\Users\runneradmin\AppData\Local\uv\cache\builds-v0\.tmpxJp0XV\lib\site-packages\setuptools\build_meta.py", line 497, in run_setup
    super().run_setup(setup_script=setup_script)
  File "C:\Users\runneradmin\AppData\Local\uv\cache\builds-v0\.tmpxJp0XV\lib\site-packages\setuptools\build_meta.py", line 313, in run_setup
    exec(code, locals())
  File "<string>", line 24, in <module>
  File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\encodings\cp1252.py", line [23](https://github.com/drivendataorg/deon/actions/runs/9822962789/job/27168832304#step:4:24), in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x8d in position 4697: character maps to <undefined>

This is happening when setuptools is trying to run our setup.py. Line 24 of setup.py is:

deon/setup.py

Line 24 in 4f1fc6b

long_description = open(Path(__file__).parent / "README.md").read()

#175 recently changed README.md. Not sure the exact culprit but there are some unicode emojis added that might be relevant.

@jayqi jayqi changed the title Failed build: tests UnicodeDecodeError for README.md when installing Jul 8, 2024
@jayqi
Copy link
Member

jayqi commented Jul 8, 2024

Fixed by 9387bdb

@jayqi jayqi closed this as completed Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant