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

[hive_flutter] Null-safety support #688

Closed
fabricio-godoi opened this issue Jun 9, 2021 · 6 comments · Fixed by #698
Closed

[hive_flutter] Null-safety support #688

fabricio-godoi opened this issue Jun 9, 2021 · 6 comments · Fixed by #698
Labels
question Further information is requested

Comments

@fabricio-godoi
Copy link

Question
Greetings,

Will hive_flutter package be updated with Null-safety in pub.dev? I check that the code is ready to support it, but the pub.dev still outdated.

Thank you!

@fabricio-godoi fabricio-godoi added the question Further information is requested label Jun 9, 2021
@hhstore
Copy link

hhstore commented Jun 17, 2021

+1

@baumths
Copy link

baumths commented Jun 17, 2021

Check out #698.

The null safety score is failing because of an issue with dart:indexed_db.

@hhstore
Copy link

hhstore commented Jun 20, 2021

Check out #698.

The null safety score is failing because of an issue with dart:indexed_db.

  • Now that this pr has been merged, when can it be published to pub.dev?
  • Does anyone handle this?
  • I really hope it will be publish soon, thanks.

@leisim @themisir

@themisir themisir linked a pull request Jun 20, 2021 that will close this issue
@hhstore
Copy link

hhstore commented Jun 21, 2021

image

  • It's weird.
    image

@baumths
Copy link

baumths commented Jun 21, 2021

Hi!

When I was working on #698 I ran pana analyzer locally, this issue is due to flutter's incompatibility with dart:indexed_db used by hive on web.

Take a look at: flutter/flutter/issues/44937

As soon as you take out the dependency on flutter from pubspec.yaml the error goes away.

@m8811163008
Copy link

don't support null safety:

  • package:hive
  • package:crypto
  • package:convert

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

Successfully merging a pull request may close this issue.

5 participants