Skip to content

Refactorings#29

Merged
finsberg merged 29 commits intomainfrom
finsberg/refactoring
Mar 9, 2026
Merged

Refactorings#29
finsberg merged 29 commits intomainfrom
finsberg/refactoring

Conversation

@finsberg
Copy link
Member

@finsberg finsberg commented Mar 6, 2026

  • Restructure the files a bit so that it is clear which commands belongs to which file types.
  • Add a lot more unit tests
  • Add documentation for script for converting between different formats
  • Add command line interface for converting between different formats

@finsberg finsberg marked this pull request as ready for review March 7, 2026 22:04
@finsberg finsberg requested a review from cdaversin March 7, 2026 22:17
Copy link
Collaborator

@cdaversin cdaversin left a comment

Choose a reason for hiding this comment

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

Looks great !! The file re-organisation makes it much clearer, with the corresponding command line (and consistent naming). And good job with all the unit tests and documentation 👏

@finsberg finsberg merged commit 430196b into main Mar 9, 2026
23 checks passed
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.

2 participants