Skip to content

Move package:native_assets_builder #69

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

Merged
merged 12 commits into from
Jun 27, 2023
Merged

Move package:native_assets_builder #69

merged 12 commits into from
Jun 27, 2023

Conversation

dcharkes
Copy link
Collaborator

@dcharkes dcharkes commented Jun 26, 2023

Closes:

And has a ton of small fixes due to the extra checks on the CI here:

  • Satisfies more lints.
  • Deleted dead code.
  • Updated dependencies.
  • Formatted files.

@github-actions github-actions bot added the type-infra A repository infrastructure change or enhancement label Jun 26, 2023
@coveralls
Copy link

coveralls commented Jun 26, 2023

Coverage Status

coverage: 98.64% (-0.9%) from 99.523%
when pulling 5024bce on native-assets-builder
into c80be90 on main.

@dcharkes dcharkes marked this pull request as ready for review June 26, 2023 19:48
@dcharkes dcharkes requested a review from HosseinYousefi June 26, 2023 19:48
@dcharkes dcharkes requested a review from devoncarew June 26, 2023 19:57
@dcharkes
Copy link
Collaborator Author

dcharkes commented Jun 26, 2023

@devoncarew Because this package is the shared implementation between dartdev and flutter_tools, and should not be used by users, we could consider hiding it in the main README.md table. What do you think? Should it be in there or not?

Edit: the test mono repo lists all packages, but provides no description for the internal/private ones: https://github.com/dart-lang/test/tree/master

Copy link
Member

@HosseinYousefi HosseinYousefi 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 @dcharkes!

@dcharkes dcharkes merged commit e12d3e6 into main Jun 27, 2023
@dcharkes dcharkes deleted the native-assets-builder branch June 27, 2023 09:18
@devoncarew
Copy link
Member

@devoncarew Because this package is the shared implementation between dartdev and flutter_tools, and should not be used by users, we could consider hiding it in the main README.md table. What do you think? Should it be in there or not?

Edit: the test mono repo lists all packages, but provides no description for the internal/private ones: https://github.com/dart-lang/test/tree/master

I guess I have mixed feelings. I'd normally say to list everything (for completeness, and to help users understand the repo contents). You'd then use the description column and or the published/version column to help indicate which packages were more user facing and which more impl. details.

There might be some types of packages that I would strip out I suppose - like test support / fixture packages.

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jun 28, 2023
The Dart SDK CL for dart-lang/native#69.

Bug: dart-lang/native#67
Change-Id: I45d7ac691a6aaa41bce5be0e36403021a948bb4f
Cq-Include-Trybots: luci.dart.try:pkg-linux-debug-try,pkg-linux-release-try,pkg-mac-release-arm64-try,pkg-mac-release-try,pkg-win-release-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/311740
Reviewed-by: William Hesse <[email protected]>
Commit-Queue: Daco Harkes <[email protected]>
Reviewed-by: Devon Carew <[email protected]>
HosseinYousefi added a commit that referenced this pull request Nov 16, 2023
Added support for Java arrays. Closed #69.

* `jni`: Added `JniArray<T>` and `JniType<T>` that includes signature information about Java types.

* `jnigen` generates an extension for `JniArray<GeneratedClass>` so we can use `operator []` and `operator []=` on all arrays. `jni` types also have this extension.

* Arrays also have a `setRange` method similar to `List`s.

* `jnigen` generates a private type class that extends `JniType<GeneratedClass>` accessible through `.type` static getter. This is useful for creating arrays since they require a type parameter like `JniArray(JniString.type, 5)` which creates an array of `JniString` with a length of 5.
HosseinYousefi added a commit that referenced this pull request Nov 16, 2023
Added support for Java arrays. Closed #69.

* `jni`: Added `JniArray<T>` and `JniType<T>` that includes signature information about Java types.

* `jnigen` generates an extension for `JniArray<GeneratedClass>` so we can use `operator []` and `operator []=` on all arrays. `jni` types also have this extension.

* Arrays also have a `setRange` method similar to `List`s.

* `jnigen` generates a private type class that extends `JniType<GeneratedClass>` accessible through `.type` static getter. This is useful for creating arrays since they require a type parameter like `JniArray(JniString.type, 5)` which creates an array of `JniString` with a length of 5.
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package:native_assets_cli type-infra A repository infrastructure change or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants