-
Notifications
You must be signed in to change notification settings - Fork 9
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
Use generator instead of transformer #9
Comments
Thanks for your comment. I have never wrote a transformer but I am eager to learn. Do you mean that this is the Builder API: https://github.com/dart-lang/build ? |
Yep. This is it. Example of the generator: Generator must be separate package. This prevent problems from analyzer, source_gen deps. Because they always must be sync with upstream. |
Thanks a lot for your issue. I will have a look at this during the week 👍 . |
Hi @Aetet, I am looking at Generators right now and I have some questions.
Thanks in advance! EDIT: Ready almost ready! https://github.com/hourliert/graphql_client/tree/master/graphql_client_generator 😂 |
Hi, @hourliert
And if your package for coverage So you need downgrade angular, or patch dart_dev by yourself, or separate dart_dev as external package. And run it like cli. Occasionally will run into this problem. For built_value_generator, dartfmt, less_transformer, etc. That's why you need to reduce it with separate package. To solve this problem by infrastructure we need separate section at pubspec for this deps. But AFAIK it won't be shipped at Dart 2.0.
I think good practice to use single repo for generator and library, but have separate package for them at pub. Thanks to you! |
Awesome, thanks a lot for your answer. |
@hourliert Has there been any progress on this front? This would indeed be a handy feature! I'd love to help out where I can. |
As I know dart team will deprecate transformer API favor to builder api. Like https://github.com/google/built_value.dart I think it would be great if it ll be builder from the beginning
The text was updated successfully, but these errors were encountered: