-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Fixes for Dart imports #8380
base: master
Are you sure you want to change the base?
Fixes for Dart imports #8380
Conversation
I'm not a maintainer so I don't have merge permissions |
@aardappel Sorry to tag you out of the blue. Can you promote the merger of PR? |
This was merged: #8313 Can you rebase this one, seems there may be some conflicts. |
There also seem to be dart codegen changes to the generated files that are not included in this PR |
4c76b91
to
115c8e4
Compare
@aardappel thanks for the review - I've rebased and updated the generated bindings. Happy to squash this down before merge. |
This still seems to be an issue |
Fixes #7609
Fixes #8379
This PR builds on the work from #8259 to:
export
statement to matchimport
statements for files in the same namespace to ensure that include chains work in the generated code