-
Notifications
You must be signed in to change notification settings - Fork 38
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
Migrate code excerpter packages to null safety #146
Migrate code excerpter packages to null safety #146
Conversation
8e2f6e2
to
2b4bce0
Compare
@RedBrogdon Can you review this or assign someone on the devrel team? No rush as I'm waiting on the infrastructure changes to be complete first. @kevmoo Could you also take a look if you get a chance? Thanks! 💙 |
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.
in general, LGTM – careful about changes not related to the PR intent.
c3ed387
to
2fe6d5f
Compare
2fe6d5f
to
64e23d7
Compare
Hey @parlough! Sorry I missed this PR. I'll give it a review by EOD tomorrow! |
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.
Null safety changes LGTM! Thank you @parlough!
Though, this is my first time reviewing such a large chunk of Dart code! @johnpryan @domesticmouse can either of you please give this a quick once-over to make sure I didn't miss anything? Thank you!
Can someone squash and merge this? I don't have commit access to site-shared. Thanks! |
Updates the code excerpter packages to the null safety. Cleans up some code with new language features and standards along the way.
Fixes #143
For review: @khanhnwin