-
Notifications
You must be signed in to change notification settings - Fork 75
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
Feature Request: Sorted Imports #918
Comments
mmmh not sure as the order of import does impact the bytecode generated and could potentially affect the way the state is stored and the amount of gas it uses on different calls. doesn't feel right with me even as an @fvictorio any thoughts? |
@Janther Can you explain to me how import order impacts the bytecode? From my (albeit limited) experimenting, it seems to have no affect. Thanks! |
I think I was mistaken, I was thinking of the order of inheritance in a contract. |
Parking this for now, it seems to add some complexity (specially if we add comments for the imports into the mix). |
I am of the opinion that this kind of thing belongs in a linter (tagging @juanpcapurro, who is maintaining That being said, not everyone agrees with that point of view. For example, |
I would love if there was an option to sort imports alphabetically. Liked groups could be segregated with a space (ex the way
go fmt
works).The text was updated successfully, but these errors were encountered: