Skip to content

Conversation

@dcharkes
Copy link
Contributor

@dcharkes dcharkes commented May 21, 2025

The experiment flag will be on by default so it is not passed as argument anymore.

Accompanying Dart CL: https://dart-review.googlesource.com/c/sdk/+/429920

@dcharkes dcharkes requested a review from a team as a code owner May 21, 2025 09:45
@github-actions
Copy link

PR Health

Changelog Entry ✔️
Package Changed Files

Changes to files need to be accounted for in their respective changelogs.

@dcharkes
Copy link
Contributor Author

FYI @mosuem It looks like this publish lint violation is intentional.

Copy link
Contributor

@goderbauer goderbauer left a comment

Choose a reason for hiding this comment

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

LGTM

@dcharkes dcharkes merged commit 42a6d33 into master May 21, 2025
58 of 59 checks passed
@dcharkes dcharkes deleted the native-assets-graduate branch May 21, 2025 10:00
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request May 21, 2025
This PR enables native assets on the main and dev channel by default,
and make native assets available on the beta channel.

This PR removes the flag from invocations.

The helper packages (`package:hooks` and `package:code_assets`) will
stay 0.x for now, until the SDK constraint can be bumped to a beta
release and we're happy with the Dart API.

`dart build` is also made available as preview (without a flag on
the main, dev, and beta channels). We're still finalizing the spec for
this command. (#60730)

`dart test` will need dart-lang/test#2501.
This means users will need to update their `package:test` dependency.

This PR refactors the way that invalid `package_config.json`s are
handled: they are now loaded in the dartdev commands and handled there.

Bug: #50565
Project: https://github.com/orgs/dart-lang/projects/99/

Change-Id: I7db9ff6d7196750cab9379a4605c6bbf89a974d7
Cq-Include-Trybots: luci.dart.try:pkg-linux-debug-try,pkg-linux-release-arm64-try,pkg-linux-release-try,pkg-mac-release-arm64-try,pkg-win-release-arm64-try,pkg-mac-release-try,pkg-win-release-try
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/429920
Commit-Queue: Daco Harkes <[email protected]>
Reviewed-by: Martin Kustermann <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants