Skip to content

Apply small fixes - #9

Merged
vietle-bh merged 3 commits into
developfrom
SmallFixes
Nov 2, 2026
Merged

vietle-bh merged 3 commits into
developfrom
SmallFixes

Conversation

@vietle-bh

@vietle-bh vietle-bh commented Nov 2, 2026 •

Copy link
Copy Markdown
Collaborator

NOTE: Depends on

Issues addressed by this PR

This pull request introduces several improvements to the Python schema generation process, focusing on better handling of missing or redundant assembly folders, improved command-line options, and enhanced test coverage. The most important changes are as follows:

Assembly Folder Cleanup and Reporting:

  • Added a new --clean-missing-assemblies command-line flag and supporting logic to detect and optionally delete assembly folders that were not generated in a full run, preventing old files from persisting unnoticed. If the flag is not set, a warning is issued instead. [1] [2] [3] [4] [5] [6] [7]

File Output Safety:

  • Improved file writing logic to detect and prevent silent overwrites when two classes would generate the same file (case-insensitive), throwing an exception with details about the conflict. [1] [2]

Test Enhancements:

  • Added tests to ensure the --clean-missing-assemblies flag is parsed correctly and behaves as expected, and to verify that generated file headers do not include a year range, preventing unnecessary diffs when the year changes. [1] [2]

Generated Code Improvements:

  • Updated the copyright header in generated Python files to remove the year range, ensuring stable diffs across years.

Fixture and Type Handling:

  • Ensured that DateTimeOffset properties are handled correctly in generated code and verified via unit tests. [1] [2]

Test files

Changelog

Additional comments

@vietle-bh vietle-bh self-assigned this Nov 2, 2026
@vietle-bh
vietle-bh merged commit be5cbb0 into develop Nov 2, 2026
3 checks passed
@vietle-bh
vietle-bh deleted the SmallFixes branch November 2, 2026 12:30
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