migrate to null-safety

[morvagergely]

This commit migrates all packages to null-safety. Solves #564.
Also updates dependencies.
Android compile SDK version changed from 28 to 29 and multidex support has been enabled.
queryRenderedFeatures now works on web.

[mendoxe]

Hi, could this issue get more priority, please? It is the last not null-safe package for my project and I would like to migrate. Thanks in advance!

[andrea689]

@mendoxe If you want to migrate use this:

dependency_overrides:
  mapbox_gl:
    git:
      url: https://github.com/morvagergely/flutter-mapbox-gl.git
  mapbox_gl_platform_interface:
    git:
      url: https://github.com/morvagergely/flutter-mapbox-gl.git
      path: mapbox_gl_platform_interface
  mapbox_gl_web:
    git:
      url: https://github.com/morvagergely/flutter-mapbox-gl.git
      path: mapbox_gl_web

[mendoxe]

@andrea689 Thank you, sir.

Btw. may I ask why isn't the package being migrated and one has to do a workaround like this?

[andrea689]

@mendoxe This is a community driven project, so the PR will be merged and a new version released when the maintainer has some time to do it. In the meantime, you can use the fork directly and proceed with the migration.

[n8han]

Thanks for doing this! I've tested this branch with my own app and haven't run into any issues. And it's nice to see a few latent bugs being fixed along the way.

I have a few comments and questions here.

Also, for other reviewers, I did not look at changes to examples and the web platform code. Someone who know the web codebase may want to take a closer look at that.

[n8han]

lgtm, but as I mentioned I didn't review the web platform code

[andrea689]

I've tried my apps (Android iOS and web) with this branch and there doesn't seem to be any problems.

@tobrun I think you should merge this PR as soon as possible so that the next developments can be done on an NNBD basis.

[hpoul]

Any plan on releasing a new version to pub which includes this pr? @andrea689 ?

[andrea689]

@tobrun makes the releases

[tobrun]

@tobrun makes the releases

yes, I need to find a way for collaborators to be able to do this as well. Initially thinking to add collaborators with write access, now thinking about setting up a CD pipeline through git tags..

[supposedlysam-bb]

Would appreciate any update on this @tobrun. Were you able to set up a CD pipeline or manually release it to the pub?

[pitazzo]

Also interested into getting this PR released!

[DmitriyYakovlev]

any news regarding null-safety release?