Skip to content

fold in feedback from e2e document review#36

Merged
chgray merged 13 commits intomainfrom
user/chgray/address_feedback
Jun 23, 2025
Merged

fold in feedback from e2e document review#36
chgray merged 13 commits intomainfrom
user/chgray/address_feedback

Conversation

@chgray
Copy link
Copy Markdown
Contributor

@chgray chgray commented Mar 18, 2025

Address some feedback from a (quick) e2e review - note this doesnt push any document beyond the (low) ReviewLevel1b bar

Changes

update a few documents to be clearer, add forgotten portions, and clean up a few URL links

Merge requirement checklist

  • CONTRIBUTING guidelines followed.
  • Unit tests added/updated (if applicable).

@chgray chgray enabled auto-merge (squash) March 18, 2025 17:34
Comment thread docs/docs/Architecture.KeyConstructs.Overview.document.md
Comment thread docs/docs/Architecture.KeyConstructs.Overview.document.md Outdated
Co-authored-by: Reiley Yang <reyang@microsoft.com>
Comment thread docs/docs/Architecture.KeyConstructs.Overview.document.md Outdated
Comment thread docs/docs/Architecture.KeyConstructs.Overview.document.md
Co-authored-by: Reiley Yang <reyang@microsoft.com>
@chgray chgray disabled auto-merge March 18, 2025 19:48
@chgray chgray enabled auto-merge (squash) March 19, 2025 21:43
@chgray chgray requested review from jmacd and reyang March 19, 2025 21:43
Comment thread docs/docs/Architecture.KeyConstructs.Overview.document.md Outdated
@chgray chgray merged commit ef68fcf into main Jun 23, 2025
4 checks passed
@chgray chgray deleted the user/chgray/address_feedback branch June 23, 2025 18:59
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