Skip to content

Clean-up assign-example-ids#3612

Closed
MatthiasWiesmann wants to merge 3 commits intomainfrom
main_program
Closed

Clean-up assign-example-ids#3612
MatthiasWiesmann wants to merge 3 commits intomainfrom
main_program

Conversation

@MatthiasWiesmann
Copy link
Contributor

The assign-example-ids.py script was stand-alone python script that was execed from the main script.

  • Move the code into an actual function.
  • Instead of going all the examples twice, keep track of the changed examples and their path in a dictionary.
  • Perform atomic rewrite of example files.
  • Renamed to assign_example_ids so it can be a package.
  • Call this package's function from the main script.
  • Use logging statements consistently.

@MatthiasWiesmann MatthiasWiesmann deleted the main_program branch September 16, 2024 08:36
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