Skip to content

release: Fix crash on flutter run and upgrade to flutter 3.29.3#285

Merged
taka-okayama merged 5 commits intosony:mainfrom
zxcv1884:main
Jul 9, 2025
Merged

release: Fix crash on flutter run and upgrade to flutter 3.29.3#285
taka-okayama merged 5 commits intosony:mainfrom
zxcv1884:main

Conversation

@zxcv1884
Copy link
Contributor

@zxcv1884 zxcv1884 commented Jun 2, 2025

Description:

  • Fixed UnimplementedError from provideVmService by replacing it with an empty method to prevent crash during flutter-elinux run on eLinux.
  • Upgraded Flutter to 3.29.3 since 3.29.2 artifacts were removed from GitHub.
  • Added new author for consistency.

zxcv1884 added 2 commits June 2, 2025 11:58
The release artifacts for Flutter 3.29.2 have been removed from GitHub
flutter run was crashing on eLinux because provideVmService threw UnimplementedError.

Replaced it with an empty async method body (`{}`) to prevent the crash.
@zxcv1884
Copy link
Contributor Author

@taka-okayama Could you take a look at this PR? Thanks!

@showier-drastic
Copy link

Your pull request seems to have inconsistent formatter line length settings with the original code. This causes many code that you do not intend to touch to appear in your changeset. You need to rework the PR to change the formatter settings to align with the original code, so that unchange code will not be included in the commit.

@zxcv1884
Copy link
Contributor Author

Thanks! I didn’t change the formatting in the first few commits, but GitHub Actions reported formatting issues. So I followed the formatter suggestions in a later commit.

@zxcv1884
Copy link
Contributor Author

I think the formatter behavior might have changed in the new Dart version, which is why the GitHub Action failed.
If formatting isn’t needed, just let me know and I’ll revert those changes.

The `unused_element_parameter` lint warning for the `feature`
parameter was causing failures in the GitHub Actions CI workflow.
@taka-okayama taka-okayama merged commit cdc9622 into sony:main Jul 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants