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

W3D3_fix, Issue #1071 #1076

Merged
merged 2 commits into from
Jul 31, 2023
Merged

W3D3_fix, Issue #1071 #1076

merged 2 commits into from
Jul 31, 2023

Conversation

GaganaB
Copy link
Collaborator

@GaganaB GaganaB commented Jul 31, 2023

Fixing Issue #1071

@github-actions
Copy link

W3D3_Tutorial1
Instructor Open In Colab
Student Open In Colab
Code report for W3D3_Tutorial1

Code report for W3D3_Tutorial1.ipynb

Quality (pyflakes)

Total code issues: 0

Style (pycodestyle)

Total PEP8 violations: 170

Common problems:

  • 35 instances of E402 (module level import not at top of file)
  • 13 instances of E266 (too many leading '#' for block comment)
  • 12 instances of E501 (line too long (100 > 88 characters))
  • 9 instances of E501 (line too long (101 > 88 characters))
  • 9 instances of E226 (missing whitespace around arithmetic operator)
  • 8 instances of E302 (expected 2 blank lines, found 1)
  • 6 instances of E501 (line too long (91 > 88 characters))
  • 6 instances of E305 (expected 2 blank lines after class or function definition, found 1)
  • 6 instances of E127 (continuation line over-indented for visual indent)
  • 6 instances of E703 (statement ends with a semicolon)

@GaganaB GaganaB requested a review from spirosChv July 31, 2023 19:34
Copy link
Collaborator

@spirosChv spirosChv left a comment

Choose a reason for hiding this comment

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

looks good to me

@spirosChv spirosChv merged commit a0912b3 into main Jul 31, 2023
@spirosChv spirosChv deleted the w3d3_fix branch July 31, 2023 20:21
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.

3 participants