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.

Sign up for GitHub

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

Jump to bottom

Patching all issues #638

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Patching all issues #638

wants to merge 2 commits into from

Conversation

rawbytedev
Copy link

Resolved Issues

  1. Resolved Errors

    • Need some reviewing, might have missed some errors.

  2. Added Comments

    • Improved code clarity with explanatory comments.

  3. Changed Print to Return

    • Changed print statements to return to avoid None outputs.

  4. Removed <\div>

    • Cleaned up HTML by removing unnecessary <\div> elements.

  5. Reviewed and Numbered Exercises

    • Numbered exercises for easier reference.

  6. Removed Unnecessary ;

    • Although semicolons aren't used in Python, their presence doesn’t trigger an error.

  7. Reviewed Python (.py) Files

    • Ensured all Python files are reviewed and optimized.

@rawbytedev
upload fixes and changes
f65a3af 
Resolved errors(need some reviewing, might have missed some errors)
Added comments
Changed print to return (to avoid None)
Removed <\div>
Reviewed exercises and numbered them
Removed ";" in some part (; doesn't trigger error)
Reviewed the python files
@rawbytedev
uncomment
20f33cc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rawbytedev