Skip to content

Cleanup the sdotermsource library.#3597

Merged
alex-jansen merged 1 commit intomainfrom
performance_try_n
Aug 28, 2024
Merged

Cleanup the sdotermsource library.#3597
alex-jansen merged 1 commit intomainfrom
performance_try_n

Conversation

@MatthiasWiesmann
Copy link
Contributor

  • Moved all the import statements to the start and sorted them.
  • Fix spacing
  • Remove commented out code
  • Use light comprehension to shorten code and get rid of loops.
  • Replace print calls with logging.
  • Move the function to parse a single file into a protected function

@alex-jansen alex-jansen merged commit b958c70 into main Aug 28, 2024
@alex-jansen alex-jansen deleted the performance_try_n branch August 28, 2024 13:14
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

Comments