-
Notifications
You must be signed in to change notification settings - Fork 700
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
Add documentation about security advisories #5480
Conversation
Co-authored-by: Jonas Finnemann Jensen <[email protected]>
Co-authored-by: Jonas Finnemann Jensen <[email protected]>
/gcbrun |
Visit the preview URL for this PR (updated for commit 5d2363d): |
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, but some broken markdown so I left suggestions
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
Co-authored-by: Marya <[email protected]>
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.
Thanks!
Exciting stuff. Thanks all! |
Follow-up to #5480 Fixes #5458 A few things: - Adjusts the placeholder redirect to point to the new page - Add a cross-link from the pubspec reference page - Fixes a few typos - Use descriptive link text instead of "Here" - Remove `/en` from the GitHub link, so it's more likely redirect to the desired language of the user - In the "Ignoring security advisories" section, if putting the pubspec field `ignored_advisories` in code font, it's not really "a list of `ignored_advisories`, but rather a list itself. - Uses proper capitalization for "GitHub" - Use "the Dart team" instead of "we" for the recommendation - Some other minor wording adjustments _(please take a look :])_
Co-authored-by: Jonas Finnemann Jensen <[email protected]> Co-authored-by: Marya <[email protected]>
Follow-up to dart-lang#5480 Fixes dart-lang#5458 A few things: - Adjusts the placeholder redirect to point to the new page - Add a cross-link from the pubspec reference page - Fixes a few typos - Use descriptive link text instead of "Here" - Remove `/en` from the GitHub link, so it's more likely redirect to the desired language of the user - In the "Ignoring security advisories" section, if putting the pubspec field `ignored_advisories` in code font, it's not really "a list of `ignored_advisories`, but rather a list itself. - Uses proper capitalization for "GitHub" - Use "the Dart team" instead of "we" for the recommendation - Some other minor wording adjustments _(please take a look :])_
Co-authored-by: Jonas Finnemann Jensen <[email protected]> Co-authored-by: Marya <[email protected]>
Follow-up to dart-lang#5480 Fixes dart-lang#5458 A few things: - Adjusts the placeholder redirect to point to the new page - Add a cross-link from the pubspec reference page - Fixes a few typos - Use descriptive link text instead of "Here" - Remove `/en` from the GitHub link, so it's more likely redirect to the desired language of the user - In the "Ignoring security advisories" section, if putting the pubspec field `ignored_advisories` in code font, it's not really "a list of `ignored_advisories`, but rather a list itself. - Uses proper capitalization for "GitHub" - Use "the Dart team" instead of "we" for the recommendation - Some other minor wording adjustments _(please take a look :])_
No description provided.