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

feat: update unicode characters #252

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

suppo01
Copy link

@suppo01 suppo01 commented Nov 4, 2024

This PR focuses on resolving the unicode decode error run by gatorgrade on Windows.

What is the current behavior?

A unicode decode error is thrown on Windows machines.

What is the new behavior if this PR is merged?

The error will be resolved and correctly run gatorgrader checks on Windows for gatorgrade.

Other information

This PR has:
  • Commit messages that are correctly formatted
  • Tests for newly introduced code
  • Docstrings for newly introduced code

This PR is a small change
that fixes #251

Developers

@suppo01

@suppo01 suppo01 marked this pull request as draft November 4, 2024 16:02
@AlishChhetri AlishChhetri self-requested a review November 7, 2024 20:09
@AlishChhetri
Copy link
Collaborator

@suppo01, could you please investigate why most of the checks failed? Understanding any specific issues or configuration challenges would be helpful. While the PR introduces a small but likely impactful change, we’ll need to resolve these issues before proceeding. Thank you!

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.

Unicode Characters in the constants.py file are not compatible with Windows
2 participants