Skip to content

Conversation

@michaeltlombardi
Copy link
Contributor

PR Summary

Adds an edited draft of the output accessibility document for DSC v3.

PR Checklist

  • Descriptive Title: This PR's title is a synopsis of the changes it proposes.
  • Summary: This PR's summary describes the scope and intent of the change.
  • Contributor's Guide: I have read the contributors guide.
  • Style: This PR adheres to the style guide.

@learn-build-service-prod

This comment was marked as outdated.

Copy link
Contributor

@sdwheeler sdwheeler left a comment

Choose a reason for hiding this comment

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

Two minor changes

Copy link
Member

@theJasonHelmick theJasonHelmick left a comment

Choose a reason for hiding this comment

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

With Sean's changes - LGTM

@learn-build-service-prod
Copy link

Learn Build status updates of commit 36dc3e1:

✅ Validation status: passed

File Status Preview URL Details
dsc/docs-conceptual/dsc-3.0/concepts/output-accessibility.md ✅Succeeded View (dsc-3.0)
dsc/docs-conceptual/dsc-3.0/toc.yml ✅Succeeded View (dsc-3.0)

For more details, please refer to the build report.

For any questions, please:

Copy link
Contributor

@mgreenegit mgreenegit left a comment

Choose a reason for hiding this comment

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

Looks good. :shipit:

@sdwheeler sdwheeler merged commit 8bcb6cb into MicrosoftDocs:main Oct 29, 2024
3 checks passed
sdwheeler added a commit that referenced this pull request Oct 29, 2024
* (AB#330346) Add doc for output accessibility

* Apply suggestions from code review

---------

Co-authored-by: Mikey Lombardi (He/Him) <mlombardi@microsoft.com>
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.

4 participants