We've standardized on importing specific types, vs. the whole file (which sometimes "hides" dependencies, as it also pulls in that file's imports). The exception is VaultTypes, when the list gets too long (multi-line), and impairs readability. (Except in examples, where it also serves as documentation.)
A few were still done in the old style.
Type of change
[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Dependency changes
[X] Code refactor / cleanup
[ ] Optimization: [ ] gas / [ ] bytecode
[ ] Documentation or wording changes
[ ] Other
Checklist:
[X] The diff is legible and has no extraneous changes
[N/A] Complex code has been commented, including external interfaces
[ ] Tests have 100% code coverage
[X] The base branch is either main, or there's a description of how to merge
Description
We've standardized on importing specific types, vs. the whole file (which sometimes "hides" dependencies, as it also pulls in that file's imports). The exception is VaultTypes, when the list gets too long (multi-line), and impairs readability. (Except in examples, where it also serves as documentation.)
A few were still done in the old style.
Type of change
Checklist:
main
, or there's a description of how to mergeIssue Resolution