-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
fix: issue with JSON conversion of packagings data #762
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @peterwvj!
The code itself looks good, but you removed important ignore
statements in generated files. Please put them back so that the code analyze works.
And consider my other comments ;)
@monsieurtanuki thank you for your great PR feedback. I've implemented all of your suggestions in commit df353be |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @peterwvj for this PR and the changes after my review!
Fixes #752
After adding the proposed fix, I did run into some issues regenerating the *.g.dart files using build_runner but I managed to fix that by upgrading the SDK dependencies according to the instructions described here.