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

Unicode Characters in the constants.py file are not compatible with Windows #251

Open
suppo01 opened this issue Oct 27, 2024 · 0 comments · May be fixed by #252
Open

Unicode Characters in the constants.py file are not compatible with Windows #251

suppo01 opened this issue Oct 27, 2024 · 0 comments · May be fixed by #252
Labels

Comments

@suppo01
Copy link

suppo01 commented Oct 27, 2024

Describe the bug

The unicode characters in the constants.py file need to be updated as they are not currently compatible with the Windows OS. Because of this issue, when gatorgrade uses gatorgrader, a unicode decode error is thrown which causes the gatorgrader checks to fail.

To Reproduce

Steps to reproduce the behavior:

  1. Get inside a repo where gatorgrade can be run on a Windows OS
  2. Insure a recent version of gatorgrade is installed
  3. Go where the gatorgrade.yml file can be accessed
  4. Type gatorgrade into the terminal and hit enter

Expected behavior

A unicode decode error should be thrown by your computer in the yellow diagnostic test under the failures for gatorgrader checks.

Environment

I ran the code using gatorgrade and python. Poetry was present in the repo and ran execexam commands, but it is not related to the bug I described above.

Additional context

I determined that gatorgrade is responsible because when the import for gator, aka gatorgrade, is removed from the file, the unicode decode error is no longer thrown.

@suppo01 suppo01 added the bug label Oct 27, 2024
@suppo01 suppo01 linked a pull request Nov 4, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant