Skip to content
This repository has been archived by the owner on Jan 26, 2021. It is now read-only.

Migrate to null safety #90

Merged
merged 8 commits into from
Jan 18, 2021
Merged

Conversation

davidmorgan
Copy link
Contributor

Fix #85

Not yet publishable due to unpublished deps / resolution of deps, but otherwise looks good, migration went smoothly. TGP looks good.

There is some very old style here; I have a followup PR ready applying pedantic lints+fixes :)

@davidmorgan
Copy link
Contributor Author

There's a failure for a js test, taking a look.

lib/auth.dart Outdated Show resolved Hide resolved
lib/auth.dart Outdated Show resolved Hide resolved
lib/auth_browser.dart Outdated Show resolved Hide resolved
lib/auth_browser.dart Outdated Show resolved Hide resolved
lib/auth_io.dart Show resolved Hide resolved
lib/src/crypto/pem.dart Outdated Show resolved Hide resolved
lib/src/http_client_base.dart Outdated Show resolved Hide resolved
lib/src/oauth2_flows/jwt.dart Outdated Show resolved Hide resolved
test/crypto/asn1_test.dart Show resolved Hide resolved
test/oauth2_test.dart Show resolved Hide resolved
@davidmorgan
Copy link
Contributor Author

Thanks, PTAL.

@davidmorgan davidmorgan merged commit 30c084b into dart-archive:master Jan 18, 2021
@davidmorgan davidmorgan deleted the null-safe branch January 18, 2021 11:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

Migrate to null safety
2 participants