-
Notifications
You must be signed in to change notification settings - Fork 26
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
Update dependencies and switch to nullsafety #24
Conversation
We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google. ℹ️ Googlers: Go here for more info. |
Due to the nature of the package the actual code is already nullsafe. The only changes necessary are unit tests checking for throwing errors on null.
@googlebot I fixed it. |
cc @kevmoo |
@mit-mit – should I land this? |
Yes, I think that would be great. There are fair number of dependencies of this. |
This is not used by the SDK – but in general we sync to google3 before
publishing
…On Wed, Nov 25, 2020 at 7:45 AM Michael Thomsen ***@***.***> wrote:
Yes, I think that would be great. There are fair number of dependencies of
this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#24 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAEFCSLDI4VWR7DURE3PWLSRURA7ANCNFSM4T56NIFA>
.
|
@kevmoo Just realised I missed to update the CHANGELOG.md accordingly. |
Please do! |
@kevmoo Any word on when this will land as a prerelease? |
I can get this out today! |
Published! |
Nice! Thanks :) |
Due to the nature of the package the actual library code is already nullsafe and unchanged.
The only necessary changes are
2.2
.