Skip to content
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

Ban pointless Unicode characters from problem statements #1960

Merged
merged 2 commits into from
Jul 18, 2022

Conversation

int-y1
Copy link
Contributor

@int-y1 int-y1 commented Jun 24, 2022

  • i saw in a problem statement
  • i saw a problem that said Output ... `−1` if it is not possible.

@codecov-commenter
Copy link

codecov-commenter commented Jun 24, 2022

Codecov Report

Merging #1960 (80b5bde) into master (8eed5e5) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1960   +/-   ##
=======================================
  Coverage   46.46%   46.46%           
=======================================
  Files         236      236           
  Lines       13114    13114           
=======================================
  Hits         6093     6093           
  Misses       7021     7021           
Impacted Files Coverage Δ
dmoj/settings.py 97.70% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8eed5e5...80b5bde. Read the comment docs.

@int-y1 int-y1 changed the title Ban ligatures from problem statements Ban pointless Unicode characters from problem statements Jun 24, 2022
Copy link
Member

@Xyene Xyene left a comment

Choose a reason for hiding this comment

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

🙏

@Xyene Xyene merged commit 7436f42 into DMOJ:master Jul 18, 2022
@int-y1 int-y1 deleted the ban-ligature branch July 25, 2022 06:45
# 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.

3 participants