From a1a3554447e6ec592a5ecf041767db26b960255a Mon Sep 17 00:00:00 2001 From: Daco Harkes Date: Wed, 28 Feb 2024 09:16:08 +0100 Subject: [PATCH] address comments --- pkgs/native_assets_cli/example/local_asset/build.dart | 1 + pkgs/native_assets_cli/lib/native_assets_cli.dart | 1 - pkgs/native_assets_cli/lib/src/api/build.dart | 1 + pkgs/native_assets_cli/lib/src/api/build_config.dart | 3 ++- 4 files changed, 4 insertions(+), 2 deletions(-) diff --git a/pkgs/native_assets_cli/example/local_asset/build.dart b/pkgs/native_assets_cli/example/local_asset/build.dart index 36306a060..71365fe63 100644 --- a/pkgs/native_assets_cli/example/local_asset/build.dart +++ b/pkgs/native_assets_cli/example/local_asset/build.dart @@ -32,6 +32,7 @@ void main(List args) async { } output.addAsset( + // TODO: Change to DataAsset once the Dart/Flutter SDK can consume it. CCodeAsset( id: 'library:$packageName/asset.txt', file: assetPath, diff --git a/pkgs/native_assets_cli/lib/native_assets_cli.dart b/pkgs/native_assets_cli/lib/native_assets_cli.dart index 5301ad877..9369cb1b0 100644 --- a/pkgs/native_assets_cli/lib/native_assets_cli.dart +++ b/pkgs/native_assets_cli/lib/native_assets_cli.dart @@ -23,4 +23,3 @@ export 'src/api/ios_sdk.dart' show IOSSdk; export 'src/api/link_mode.dart' show LinkMode; export 'src/api/link_mode_preference.dart' show LinkModePreference; export 'src/api/os.dart' show OS; -// export 'src/api/target.dart' show Target; diff --git a/pkgs/native_assets_cli/lib/src/api/build.dart b/pkgs/native_assets_cli/lib/src/api/build.dart index 5b8759c02..50fd43dfb 100644 --- a/pkgs/native_assets_cli/lib/src/api/build.dart +++ b/pkgs/native_assets_cli/lib/src/api/build.dart @@ -68,6 +68,7 @@ import 'build_output.dart'; /// } /// /// output.addAsset( +/// // TODO: Change to DataAsset once the Dart/Flutter SDK can consume it. /// CCodeAsset( /// id: 'library:$packageName/asset.txt', /// file: assetPath, diff --git a/pkgs/native_assets_cli/lib/src/api/build_config.dart b/pkgs/native_assets_cli/lib/src/api/build_config.dart index 309bbcf8e..f480703c0 100644 --- a/pkgs/native_assets_cli/lib/src/api/build_config.dart +++ b/pkgs/native_assets_cli/lib/src/api/build_config.dart @@ -90,7 +90,8 @@ abstract final class BuildConfig { /// Not available during a [dryRun]. CCompilerConfig get cCompiler; - /// Whether the current run is a "dry run". + /// Indicates whether a run should build (wet run) or report potential assets + /// (dry run). /// /// If so, the build won't actually be run, but will report the native assets /// which would have been produced.