maRci002

Results 112 comments of maRci002

I've addressed your comments but removing those two `dependency_overrides` section results error like this: > Error (Xcode): ../lib/video_player.dart:441:29: Error: Member not found: 'isPlayingStateUpdate'.

How can add back the reviewers?

@reidbaker I fixed the tests. I still need #3361 PR to be merged according to https://github.com/flutter/flutter/wiki/Contributing-to-Plugins-and-Packages#changing-federated-plugins

In my opinion this PR is ready - removed `dependency_overrides` - removed `// ignore: missing_enum_constant_in_switch` which was created by #3435 - updated `video_player_platform_interface` dependencies to `6.1.0` since #3361 PR increased...

It seems flutter/engine#34085 PR did fix the issue it is available in Flutter 3.7.0 stable release.

Thanks, @justinmc. I've just created the PR for the framework part flutter/flutter#141373

In my opinion, this PR is ready for further review. However, I've encountered some issues with testing, particularly related to the `@Config` annotation. I'm currently awaiting some inputs from @camsim99...

> @maRci002 Looks like tests are passing! Is this ready for re-review? @camsim99, thanks, but the API might change over time. I will definitely let you know when it's ready...

> I think once you've fixed the merge conflict we should move forward with merging this PR, and then focus on getting the framework PR merged. Framework and Engine should...