Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review code executed at import time #955

Open
Tracked by #1085
lbianchi-lbl opened this issue Aug 31, 2021 · 1 comment
Open
Tracked by #1085

Review code executed at import time #955

lbianchi-lbl opened this issue Aug 31, 2021 · 1 comment
Assignees
Labels
backlog Might get to it someday Priority:Normal Normal Priority Issue or PR

Comments

@lbianchi-lbl
Copy link
Contributor

It would be useful to do a review or what code is executed when importing a particular module across the complete codebase.

Generally, we'd probably want to keep import-level code as minimal as possible; otherwise, the chance of introducing unexpected behavior or bugs that are difficult to detect will be higher.

@lbianchi-lbl lbianchi-lbl added the Priority:Normal Normal Priority Issue or PR label Aug 31, 2021
@lbianchi-lbl lbianchi-lbl self-assigned this Aug 31, 2021
@lbianchi-lbl
Copy link
Contributor Author

This is related to #1013, in that making sure that any given module can be imported "cleanly" requires some way of gracefully handling unavailable modules imported there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Might get to it someday Priority:Normal Normal Priority Issue or PR
Projects
None yet
Development

No branches or pull requests

2 participants