Skip to content

Conversation

@davidmorgan
Copy link
Contributor

@davidmorgan davidmorgan commented Sep 10, 2025

Non-doc changes

  • allow empty package or path in AssetId.parse since the main constructor allows them, add test coverage showing it allows them
  • tweaks to error messages
  • add test coverage to changeExtension so I am sure I documented it correctly :)
  • rename test for asset_id.dart: id_test.dart -> asset_id_test.dart

Doc / readability improvements

  • rewrite for clarity / brevity
  • some corrections about which exceptions can be thrown
  • move members around to put most important ones
  • move AssetReader and AssetWriter next to BuildStep and the docs on BuildStep since that's the class builders interact with directly
  • add docs to BuilderFactory and BuildConfig

I didn't get to all the API yet, more to come later :)

@github-actions
Copy link

github-actions bot commented Sep 10, 2025

PR Health

Changelog Entry ✔️
Package Changed Files

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

This check can be disabled by tagging the PR with skip-changelog-check.

Copy link
Contributor Author

@davidmorgan davidmorgan left a comment

Choose a reason for hiding this comment

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

Thanks :)

@davidmorgan davidmorgan merged commit 01da402 into dart-lang:master Sep 10, 2025
18 checks passed
@davidmorgan davidmorgan deleted the build-api-docs branch September 10, 2025 12:10
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.

2 participants