Skip to content

Conversation

@Zhigal
Copy link
Contributor

@Zhigal Zhigal commented Jan 26, 2024

Sorry for a HUGE pr...

In this PR:

  • PEP8 formatting applied:
    • docstring converted to """ from '''
    • emply lines added where needed
    • self replaced with cls for class methods
  • proofreading
  • mutable arguments for some methods were replaced with immutable
  • unused imports removed
  • None comparison replaces with is None check
  • unnecessary parenthesis removed
  • array initialization optimized for this cases:
arr = []
arr.append(...)

The logic is still the same, no changes here

@Zhigal Zhigal force-pushed the fix-formatting-and-other-issues branch from 5d674c9 to d7ff690 Compare January 26, 2024 12:34
@Zhigal Zhigal force-pushed the fix-formatting-and-other-issues branch from d7ff690 to d48a2d4 Compare January 26, 2024 12:39
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.

1 participant