[go_router_builder] Adds support for required $extra parameter

[dancamdev]

This PR includes support for having type-safe non-nullable $extra parameters when working with TypedGoRoutes. Optional $extra is still supported so this isn't a breaking change.

@TypedGoRoute<RequiredExtraRoute>(path: '/requiredExtra')
class RequiredExtraRoute extends GoRouteData {
  const RequiredExtraRoute({required this.$extra});

  final Extra $extra;

  @override
  Widget build(BuildContext context, GoRouterState state) =>
      RequiredExtraScreen(extra: $extra);
}

List which issues are fixed by this PR. You must list at least one issue.
#117261

Pre-launch Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I read the Tree Hygiene wiki page, which explains my responsibilities.
• I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/packages repo does use dart format.)
• I signed the CLA.
• The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
• I listed at least one issue that this PR fixes in the description above.
• I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
• I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
• I updated/added relevant documentation (doc comments with ///).
• I added new tests to check the change I am making, or this PR is test-exempt.
• All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

[chunhtai]

Hi @Hangyujin Can you take a look at this PR?

[hannah-hyj]

LGTM

[dancamdev]

Fixed the merge conflicts with 1.2.0

[dancamdev]

Thanks @Michele-x98 for the fix!

[chunhtai]

LGTM, once the ci failure are fixed

[dancamdev]

@chunhtai should be good to go

[chunhtai]

It looks like there are still some failures

[dancamdev]

It looks like there are still some failures

Due to the following:

info - lib/src/route_config.dart:99:57 - 'element' is deprecated and shouldn't be used. Use element2 instead. Try replacing the use of the deprecated member with the replacement. - deprecated_member_use

But this is a CI issue because element2 is deprecated and element should be used instead - the go_router_builder codebase is using element across the board.

If that's not it, mind pointing me to the issue you're seeing? Thanks

[dancamdev]

@chunhtai Got it working!