-
Notifications
You must be signed in to change notification settings - Fork 52
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
Generator last version doesn't work / crash #37
Comments
Is there any breaking changes and I need to update anything else ? |
Hey, Sorry, tried to upgrade orm to Dart2 65. Flutter still uses 58. Will switch it back. |
there is a solution to this problem?... trying to test jaguar_orm_gen: ^2.1.1 from dartlang and i'm having the same issue. |
@tejainece is this just because you build it with 65 ? or is this dependency written somewhere ? Wasn't able to find it ^^ (sorry if it's a dumb question, still very new to dart) |
In Dart2, you can call constructors like functions. I replaced some such construction in calls out of practice.This has been allowed since Dart2 in preview mode. From 64 onwards, preview mode is the default mode. Now, I am stuck whether to go back or wait for Flutter to move forward. |
Will fix this tomorrow or day after |
There are lot of local changes on my side, so don't sent pull request until i push. Some new features coming up. |
You mean don't put the |
I tried to enable the preview mode but I guess it's not enough, still have problems:
Maybe because you use both relative and package import for the same file, I had the problem on my app with this and replace every import with package instead of relative, may work here too |
Or maybe is just because you use build_runner 0.9.0 where flutter can't use it
|
Yes. That too. For 65+, you need build_runner 0.9.0. I was hoping Flutter would move to 65. They are very slow. |
yeah the preview version didn't move since 50 days or something, and as they don't have any roadmap we're totally blind on that... annoying |
So, I have fixed. It also contains new features like relations without associations, etc. Pull requests are welcome. |
I mean now. Pushed all my local changes. |
Also added some unit tests. |
Yeah I need to more understand how jaguar orm / gen before being able to make PR ^^ but I will if I can for sure :)
I put back all missing
Same problem when I test on the example https://github.com/jaguar-orm/sqflite many to many it failing |
Could you send pull request with all the |
Done :) |
Opps. Sorry man. Forgot to publish jaguar_orm_gen 2.1.15. Can you try now? |
Also published your changes as |
I test with the package locally so it doesn't matter if you release or not :)
other example look ok (didn't run them but at least files are gen correctly) |
Can I see the bean spec for this many to many example? |
Do you mean this https://github.com/jaguar-orm/sqflite/tree/master/lib/many_to_many ? That's the one I test |
Alright testing it. |
WIP: Patched some of https://github.com/jaguar-orm/sqflite if you want to try with recent code. Still untested. Testing it. |
There have been some minor breaking changes in 2.1.15. Upgraded sqflite example to match them. Now, all generations work and there are no error. Will test it. |
Tested them. All seem to function like before. Let me know if I can close this issue. |
Look good to me :) I'll open a new issue if needed, the generation work so let's close this one :) |
Just try to update deps on the sqflite example and nothing is working anymore.
Here is the stack
The text was updated successfully, but these errors were encountered: