Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update for Flutter 3.10 beta #1746

Merged
merged 56 commits into from
May 10, 2023
Merged

Update for Flutter 3.10 beta #1746

merged 56 commits into from
May 10, 2023

Conversation

domesticmouse
Copy link
Contributor

@domesticmouse domesticmouse commented Apr 26, 2023

Pre-launch Checklist

  • I read the [Flutter Style Guide] recently, and have followed its advice.
  • I signed the [CLA].
  • I read the [Contributors Guide].
  • I updated/added relevant documentation (doc comments with ///).
  • All existing and new tests are passing.

domesticmouse and others added 29 commits April 26, 2023 18:21
## Pre-launch Checklist

- [x] I read the [Flutter Style Guide] _recently_, and have followed its
advice.
- [x] I signed the [CLA].
- [x] I read the [Contributors Guide].
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] All existing and new tests are passing.

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

<!-- Links -->
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[CLA]: https://cla.developers.google.com/
[Discord]: https://github.com/flutter/flutter/wiki/Chat
[Contributors Guide]:
https://github.com/flutter/samples/blob/main/CONTRIBUTING.md
@domesticmouse
Copy link
Contributor Author

@domesticmouse ffigen is now in a stable version that is Dart 3 compatible: https://pub.dev/packages/ffigen/versions/8.0.0

Looking at the test results looks like we'll have to temporarily disable the checks on the stable channel, until those are out?

Yup will do.

@mit-mit
Copy link
Member

mit-mit commented May 3, 2023

You might also want https://pub.dev/packages/jnigen/versions/0.4.0

Copy link
Contributor

@QuncCccccc QuncCccccc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for the help!

@domesticmouse
Copy link
Contributor Author

LGTM! Thanks for the help!

Thanks for the review!

domesticmouse and others added 3 commits May 3, 2023 16:34
This PR publishes a possible Angular - Flutter Web integration app.

## Pre-launch Checklist

- [x] I read the [Flutter Style Guide] _recently_, and have followed its
advice.
- [x] I signed the [CLA].
- [x] I read the [Contributors Guide].
- [x] I updated/added relevant documentation (doc comments with `///`).
- [x] All existing and new tests are passing.

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

<!-- Links -->
[Flutter Style Guide]:
https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo
[CLA]: https://cla.developers.google.com/
[Discord]: https://github.com/flutter/flutter/wiki/Chat
[Contributors Guide]:
https://github.com/flutter/samples/blob/main/CONTRIBUTING.md




Co-authored-by: Mark Thompson
<[email protected]>

Co-authored-by: Mark Thompson <[email protected]>
Comment on lines +96 to +97
)),
)),
Copy link
Member

@miquelbeltran miquelbeltran May 10, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It be good to add trailing comas here, but not a big deal

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ditman thoughts?

@domesticmouse domesticmouse merged commit 36e7a6a into main May 10, 2023
17 of 19 checks passed
@domesticmouse domesticmouse deleted the beta branch May 10, 2023 20:16
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.

None yet

7 participants