-
Notifications
You must be signed in to change notification settings - Fork 80
[hooks] [code_assets] Library dartdoc comments #2687
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
Conversation
PR HealthAPI leaks ✔️The following packages contain symbols visible in the public API, but not exported by the library. Export these symbols or remove them from your publicly visible API.
This check can be disabled by tagging the PR with License Headers ✔️
All source files should start with a license header. Unrelated files missing license headers
This check can be disabled by tagging the PR with |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/// application. | ||
/// | ||
/// Code assets can be added in a build hook as follows: | ||
/// [CodeAsset] can be added to the [BuildOutputBuilder] a build hook as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe:
/// [CodeAsset] can be added to the [BuildOutputBuilder] a build hook as | |
/// A [CodeAsset] can be added to the [BuildOutputBuilder] in a build hook as |
autosubmit label was removed for dart-lang/native/2687, because - The status or check suite health / breaking / run breaking health check has failed. Please fix the issues identified (or deflake) before re-applying this label. |
I believe the CI breakage is due to dart-lang/ecosystem#375, skipping check. |
Add library dartdoc comments that detail the main entry points for browsing the documentation of both packages.
Bug: