Migrate to NNBD

[pq]

As the ecosystem moves towards embracing null-safety, it'd be great to get this package migrated.

See: https://dart.dev/null-safety

Happy to help if you hit any snags!

[github-actions]

👋 Thanks for reporting! @robrbecker will take a look.

[robrbecker]

I agree! I gave it a once over a couple weeks ago but there were still dependencies that needed to upgrade first. I'll be sure to get it updated as soon as I can. Thanks for reaching out!

[robrbecker]

Looks like it's close to being unblocked. Just waiting on http

[pq]

I know folks are actively looking at http. Maybe @natebosch?

I see a tracking issue here: dart-lang/http#501

Thanks a million for taking a look @robrbecker!

[robrbecker]

The http package has a null safe release now. I'll take another shot at converting this package this week.

[pq]

Awesome. Thanks!

[robrbecker]

I put up an initial draft PR here #244
@pq would you be up for giving it a once over?

[pq]

sorry for the slow reply! fighting a few fires here but will happily take a look in the next few days. thanks for pushing on this!

[pq]

Thanks so much for pushing on this! I've done some integrating and while I haven't dug deeply there are some spots where we're getting nullable results and ideally they'd be non-null. For example, it would be fantastic if Issue could guarantee a non-null url, title, etc. Not sure about implications.

[bsutton]

At some point can we get a version released without the -nullsafety. Google is calling for package maintainers to publish a 'pre-release' version such as 7.1.0.
Usage of the nullsaftey designation results in warnings when a dependant package attempts to publish a pre-release.

[robrbecker]

Closing this issue. Null safe versions are 8+