-
Notifications
You must be signed in to change notification settings - Fork 65
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
Add '_i' to import prefixes #160
Conversation
This matches old behavior and is a little more clear than just a digit. This will also satisfy the lint on the latest SDK. See dart-lang/linter#684 (comment)
ping @matanlurey - is this worth getting in? |
Sorry I missed this. Would you consider this a breaking change? Should it be opt-in for now? Otherwise many golden file tests will fail, for example. |
Yeah I have some golden tests that I expect to start failing after this change. I personally wouldn't consider that breaking - I think its a test that is (perhaps necessarily) relying on an implementation detail and so I'd expect it to be brittle. I'd rather not make it an option. The API to enable it becomes another part of the public surface area which is broken if/when we remove the option, and we deal with higher complexity in the mean time. I don't have strong opinions here. I know we've had customers who want these generated files to follow that lint rule, but I'd be fine with either of: |
It doesn't bother me either way. We should document that the Allocator doesn't guarantee and output as part of this PR then? |
Added a note in the Doc comment for |
@natebosch Let me know if you want a release this week. |
No rush - this change was a "may as well do it while it's on my mind" rather than a "I need this right now". |
This matches old behavior and is a little more clear than just a digit.
This will also satisfy the lint on the latest SDK. See
dart-lang/linter#684 (comment)