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

Add flake8-quotes to the linter #1763

Merged
merged 1 commit into from
Sep 8, 2021
Merged

Conversation

Riolku
Copy link
Contributor

@Riolku Riolku commented Sep 8, 2021

Also fix all existing quoting issues

Also fix all existing quoting issues
@codecov-commenter
Copy link

Codecov Report

Merging #1763 (c22c58d) into master (56c111b) will not change coverage.
The diff coverage is 34.37%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1763   +/-   ##
=======================================
  Coverage   46.20%   46.20%           
=======================================
  Files         223      223           
  Lines       12674    12674           
=======================================
  Hits         5856     5856           
  Misses       6818     6818           
Impacted Files Coverage Δ
dmoj_install_pymysql.py 0.00% <0.00%> (ø)
judge/admin/runtime.py 66.66% <ø> (ø)
judge/contest_format/atcoder.py 30.98% <0.00%> (ø)
judge/contest_format/ecoo.py 28.57% <ø> (ø)
judge/contest_format/icpc.py 28.75% <0.00%> (ø)
judge/contest_format/ioi.py 26.82% <0.00%> (ø)
judge/contest_format/legacy_ioi.py 35.59% <ø> (ø)
judge/highlight_code.py 46.15% <0.00%> (ø)
judge/management/commands/adduser.py 0.00% <ø> (ø)
judge/management/commands/makedmojmessages.py 0.00% <0.00%> (ø)
... and 20 more

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 56c111b...c22c58d. Read the comment docs.

@quantum5 quantum5 merged commit 27f5a0e into DMOJ:master Sep 8, 2021
@Riolku Riolku deleted the flake8-quotes branch September 8, 2021 03:38
@leduythuccs
Copy link
Contributor

Hi, could you share with your command to change all double quotes to single quotes?

I have a fork with huge changes :'(

# 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.

4 participants