Skip to content
This repository has been archived by the owner on Oct 17, 2024. It is now read-only.

Merge null_safety branch into master #125

Merged
merged 16 commits into from
Jul 8, 2020
Merged

Merge null_safety branch into master #125

merged 16 commits into from
Jul 8, 2020

Conversation

jakemac53
Copy link
Contributor

This is in preparation for rolling into the SDK/internal and then publishing

jakemac53 and others added 16 commits March 17, 2020 13:32
It is now statically non-null.
This did result in one api change, which I think is for the better anyways. `CancelableOperation.valueOrCancellation` now has a return type of `T?`, where it would have previously thrown at runtime if you didn't provide a cancellation value.

Also added a workaround for dart-lang/sdk#41156 in one of the tests.
* fix nnbd tests to run with strong null safety

* update travis config

* fix analyzer issues
These can now be correctly inferred after a fix in the SDK.

dart-lang/sdk#41106
Updates version number and sdk constraint.
@jakemac53 jakemac53 requested a review from natebosch July 8, 2020 15:45
@jakemac53 jakemac53 merged commit 5a43734 into master Jul 8, 2020
@jakemac53 jakemac53 deleted the null_safety branch July 8, 2020 15:57
@jakemac53 jakemac53 restored the null_safety branch July 8, 2020 15:57
@natebosch natebosch deleted the null_safety branch August 12, 2020 05:19
mosuem pushed a commit to dart-lang/core that referenced this pull request Oct 14, 2024
Migrates this package to null safety and prepares for a null safety pre-release, once we have landed the package internally and in the SDK to verify its accuracy.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants