Hello @Bhupesh-V! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file api/urls.py:

Line 20:80: E501 line too long (96 > 79 characters)
Line 37:80: E501 line too long (88 > 79 characters)
Line 76:80: E501 line too long (89 > 79 characters)

• In the file api/views.py:

Line 232:80: E501 line too long (82 > 79 characters)
Line 324:80: E501 line too long (87 > 79 characters)

• In the file utilities/judge.py:

Line 18:80: E501 line too long (83 > 79 characters)

Comment last updated at 2020-01-22 08:07:00 UTC

DeepCode's analysis on #cb5025 found:

• ❌ 0 critical issues. ⚠️ 0 warnings and 1 minor issue. ✔️ 2 issues were fixed.

💬 This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues.

☺️ If you want to provide feedback on our bot, here is how to contact us.

Codecov Report

Merging #34 into master will decrease coverage by 3.6%.
The diff coverage is 49.35%.

@@            Coverage Diff             @@
##           master      #34      +/-   ##
==========================================
- Coverage    65.8%   62.19%   -3.61%     
==========================================
  Files          20       21       +1     
  Lines         500      619     +119     
==========================================
+ Hits          329      385      +56     
- Misses        171      234      +63

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 e31e6ca...cb50259. Read the comment docs.

This pull request introduces 1 alert and fixes 1 when merging c824621 into e31e6ca - view on LGTM.com

new alerts:

• 1 for Unused import

fixed alerts:

• 1 for Unused import

This pull request fixes 1 alert when merging cb50259 into e31e6ca - view on LGTM.com

fixed alerts:

• 1 for Unused import