-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Support macros when analyzer is used as a library #56102
Comments
A hint for anyone else who's trying this: make sure you pass in Which is a good reminder that we'll have to think about different ways the code could be running. |
While I agree that we need this longer term, I think there's work that needs to be done first before we support this kind of usage. Minimally we need to do some API design work. |
tl;dr we do support macros People using
In general, |
Thanks @scheglov, it did occur to me that there are probably multiple ways to set up the analyzer as a library, which might handle macros differently today. That's fine. For the macros launch do you think it should be / will be the case that all ways to set up the analyzer support macros by default? e.g. do you think |
Bug: #56102 Change-Id: I06a53805514655fa01e962bbc98255b5e70f3ed8 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/375720 Reviewed-by: Brian Wilkerson <brianwilkerson@google.com> Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
Bug: #56102 Change-Id: Ia55b978571e412ea7b6b7c4b6cc482f731ecdc1a Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/375760 Commit-Queue: Konstantin Shcheglov <scheglov@google.com> Reviewed-by: Brian Wilkerson <brianwilkerson@google.com> Reviewed-by: Samuel Rawlins <srawlins@google.com> Reviewed-by: Bob Nystrom <rnystrom@google.com>
I noticed that ContextBuilder currently passes through null
macroSupport
by default, with no way to provide one. (Unless you cast toContextBuilderImpl
).So people using the analyzer as a library and using
ContextBuilder
will currently get no macros.As discussed we probably want macros to work with some reasonable default settings "out of the box".
So, here is an issue for that :)
The text was updated successfully, but these errors were encountered: