Release/0.7.0 #8
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- スクリプト実行関連の記述を削除 - 冗長なdocstringを簡潔化
tempfile.gettempdir()を使用してすべてのpytest一時モジュールを クリーンアップすることで、テスト間の完全な独立性を実現
- `visited` を `visited_modules` にリネーム（より明確に） - `tree` を `ast_tree` にリネーム（AST型であることを明示） - `new_module` を `reloaded_module` にリネーム（意図を明確に） - `old_keys`/`new_keys` を `old_attrs`/`new_attrs` にリネーム（正確な意味に） - `_base_module` を `_importing_module` にリネーム（意味を明確に） - `try_import_as_module` から未使用の `is_dot_import` パラメータを削除 - `to_dependencies` から未使用の `level` と `module_name` パラメータを削除 - すべてのテストを新しい命名に合わせて更新
ツリー構築時に同じモジュールへの2回目以降の依存が 誤ってスキップされるバグを防ぐためのテストを追加。
タプルベースのDependency表現を@DataClass(frozen=True)に置き換え。 FromClause/ImportClauseを関数に変換し、不要な抽象化を削減。 - domain.py: Dependency/DependencyNode新設 - from_clause/import_clause: クラス→関数 - reload_tree(): ModuleNodeから独立関数に抽出
__init__.pyで明示的にインポートされていないモジュールは、パッケージを リロードしても検出されないという設計上の制限事項をドキュメント化。
親モジュールが複数の子モジュールに対してimportlib.reload()を呼び出すケースで、 sys.modules.pop() + import_module()パターンでは2つ目以降の子モジュールの reload時に "module xxx not in sys.modules" エラーが発生する問題を修正。
READMEのファイル構成を最新に更新し、Maya対応バージョンを明記。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.