Skip to content
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

Implement null safety #93

Merged
merged 6 commits into from Mar 4, 2021
Merged

Implement null safety #93

merged 6 commits into from Mar 4, 2021

Conversation

jama5262
Copy link
Owner

@jama5262 jama5262 commented Mar 3, 2021

What does this PR do?

Implements nullsafety and resolved the following #90 issue and #89 pr

@jama5262 jama5262 linked an issue Mar 3, 2021 that may be closed by this pull request
@CalHoll
Copy link

CalHoll commented Mar 4, 2021

Please get this merged, need compatibility with my other dependencies

@jama5262 jama5262 merged commit afa69a0 into develop Mar 4, 2021
@jama5262 jama5262 deleted the null-safety branch March 4, 2021 07:48
@jama5262 jama5262 mentioned this pull request Mar 4, 2021
@SKLn-Rad
Copy link

SKLn-Rad commented Mar 4, 2021

Hey bud, what's the ETA on a release with this in?
Just a heads up, but it is currently breaking any app with the built in flutter_localizations package bundled with the flutter 2.0.0 SDK.

Happy to help build out with any tests if it speeds things up.

@jama5262
Copy link
Owner Author

jama5262 commented Mar 4, 2021

@SKLn-Rad Didn't get what you meant by what's the ETA on a release with this in? 😅, could you please explain further

And what kind of errors are you experiencing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update intl to ^0.17.0
3 participants